Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Failing string interpolation parsing case #32

Closed
Simn opened this issue Apr 8, 2016 · 1 comment
Closed

Failing string interpolation parsing case #32

Simn opened this issue Apr 8, 2016 · 1 comment

Comments

@Simn
Copy link
Collaborator

Simn commented Apr 8, 2016

class Main {
    static function main() {
        var s = "
class C {
    static inline function printClass(c:Table<String,Dynamic>, s : String) : String {
        return '{${printClassRec(c,'',s)}}';
    }
}";
        var parser = new haxeparser.HaxeParser(byte.ByteData.ofString(s), "");
        trace(parser.parse());
    }
}

From lua.Boot, currently fails with this:

C:\GitHub\haxeparser\src\haxeparser/HaxeParser.hx:495: characters 5-10 : { msg => MissingSemicolon, pos => { file => , max => 127, min => 106 } }
C:\GitHub\haxeparser\src\haxeparser/HaxeParser.hx:1068: characters 25-36 : Called from
C:\GitHub\haxeparser\src\haxeparser/HaxeParser.hx:1058: characters 11-26 : Called from
C:\GitHub\haxeparser\src\haxeparser/HaxeParser.hx:1057: lines 1057-1060 : Called from
C:\GitHub\haxeparser\src\haxeparser/HaxeParser.hx:1039: characters 13-22 : Called from
C:\GitHub\haxeparser\src\haxeparser/HaxeParser.hx:1165: characters 35-43 : Called from
C:\GitHub\haxeparser\src\haxeparser/HaxeParser.hx:1267: characters 9-15 : Called from
C:\GitHub\haxeparser\src\haxeparser/HaxeParser.hx:923: characters 17-31 : Called from
C:\GitHub\haxeparser\src\haxeparser/HaxeParser.hx:923: characters 13-31 : Called from

The related Haxe change could be this: HaxeFoundation/haxe@b94ee60

Although this was quite a while ago...

@Simn Simn closed this as completed in 5b0bf9f Apr 28, 2016
@Simn
Copy link
Collaborator Author

Simn commented Apr 28, 2016

I have ported the Haxe implementation but this is a bit of a nightmare to test properly, so there are likely gonna be some more problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant