search

HaxeCheckstyle / haxeparser

A Haxe parser for Haxe
61 stars 23 forks source link

Parsing fails on typedef extensions #31

Closed adireddy closed 8 years ago

adireddy commented 8 years ago 
typedef TypedefName = {

    > OneTypedef,
    > OtherTypedef,

}
Error: LINE - 1: Checker - Parsing failed: Unexpected >
Stacktrace: 
Called from haxeparser/HaxeParser.hx line 650
Called from haxeparser/HaxeParser.hx line 743
Called from haxeparser/HaxeParser.hx line 743
Called from haxeparser/HaxeParser.hx line 730
Called from haxeparser/HaxeParser.hx line 542
Called from haxeparser/HaxeParser.hx line 510
Called from haxeparser/HaxeParser.hx line 510
Called from haxeparser/HaxeParser.hx line 510
Called from haxeparser/HaxeParser.hx line 510
Called from haxeparser/HaxeParser.hx line 510
Called from haxeparser/HaxeParser.hx line 510
Called from haxeparser/HaxeParser.hx line 510
Called from haxeparser/HaxeParser.hx line 510
Called from haxeparser/HaxeParser.hx line 510
Called from haxeparser/HaxeParser.hx line 510
Called from haxeparser/HaxeParser.hx line 501
Called from haxeparser/HaxeParser.hx line 265
Called from checkstyle/Checker.hx line 119
Called from checkstyle/Checker.hx line 170
Simn commented 8 years ago

I guess that's one parser.ml change that I didn't port: https://github.com/HaxeFoundation/haxe/commit/606d5c79842980eea6d1d4d265a991828ae31cfe

Pull request welcome!

nadako commented 8 years ago

Sorry for offtopic, but BTW, could we allow using semicolons instead of commas as well? Because commas look a bit weird with class notation. Also, can we do last comma/semicolon optional? (e.g. typedef A = { >B })