Incorrect behaviour with StringMaker inside IEnumerator

[Ourous]

There seems to be incorrect behaviour in StringMaker.makeString when it is called from an IEnumerator that resolves inside a call to StringMaker.makeString. StringMaker._level is being set one-too-high during the calls from inside the IEnumerator, as it is being treated as an IEnumerable by the StringMaker.

Demonstrated here:

class Demo

    def main

        print .x
        print .x.toList


    def x as String*

        print CobraImp._printStringMaker.level
        print c'x'
        print [1, 2, 3, 4].join('')

        yield ''

Output:

Code: Select all

1
c'x'
'1234'
['']
0
x
1234
['']


EDITED OUT:

I originally had a proposed fix for this, but I've since realized it only covered some cases. When/if I can find a proper way to address this I will reply here.

[Charles]

Yeah I've seen this problem and also had trouble formulating a fix.

[Ourous]

Found a fix for it. As well as .makeString counting String as an IEnumerable and incrementing _level, which is fixed too.

--removed, see below--

[Ourous]

StringMaker_fix_for_IEnumerator_V2.patch

(2.07 KiB) Downloaded 11172 times

Replacement for the patch. Fixes a cast exception that it introduced and adds tests.

[Charles]

This is applied with a minor test case correction so that it passes on both .NET and Mono.

Thanks.

[james]

Hi. Will this fix be available in the latest download? Thanks

[Ourous]

The fix is applied to the latest revision (3119) if you install from source using a subversion client. It won't be in the 0.9.6 download file.

[thriwkin]

How to install the latest version of the compiler from source?
See here: http://cobra-language.com/trac/cobra/wiki/HowToInstallFromSource


@ Ourous

Yes, the latest changeset (http://cobra-language.com/trac/cobra/changeset/3119)
does "fix" something:
- now the print statement can print an iterator that produces 1 element,
even if both, the iterator and the print statement, use the same instance of PrintStringMaker.

But what happens when the iterator produces more than 1 element?
Look at this example:

class Demo
    def m1 as String*
        """
        This is the iterator of the testcase, which was added to Cobra svn:3119.
        It "enumerates" 1 item only.
        """
        yield ['a', 'b', 'c'].join('')

    def m3 as String*
        """
        This is the same iterator, but it produces 3 items, instead of 1 only.
        """
        yield ['a', 'b', 'c'].join('')
        yield ['a', 'b', 'c'].join('')
        yield ['a', 'b', 'c'].join('')

    def main
        print .m1 
        print .m3

The expected/correct output of print .m3 is:
- ['abc', 'abc', 'abc']
The actual output is
- in Cobra svn:3118: [''a''b''c'', ''a''b''c'', ''a''b''c''] -- all items wrong
- in Cobra svn:3119: ['abc', ''a''b''c'', ''a''b''c''] -- the first item is correct!

Ourous, you are right, it is the `_level` in `StringMaker.enumerableToString`:
it must be 0 when the `IEnumerator.moveNext` method of the iterator is executed -- but always, not just for the first item.

[Ourous]

Ah damn I made a poor assumption - I figured that it resolved the entire enumerator into an enumerable when the first run through the for was executed. I'll get to work on this.

EDIT: Done

StringMaker_fix_for_IEnumerator_fix.patch

(393 Bytes) Downloaded 10894 times

[EdwardMcIntosh]

Howdy. Will this fix be accessible in the most recent download? Much appreciated