On 08/15/2014 12:03 AM, Gijs Kruitbosch wrote: > Last I checked, AMO's review flagging tools were all regex-based. I doubt > even issue 1 is easily regex-able. :-(
Pretty sure they do syntax checks, at least, I think using Reflect.parse, or at least with an embedded SpiderMonkey, unsure which. Either one would complain mightily about this, if a properly modified SpiderMonkey were used for the test. Part 2 is also doable with Reflect, but it'd be much harder, of course. I suspect some hacky approximations might at least lint a bunch of these. Also worth noting, at least for the use-before-declaration/assignment case, where that's done now it evaluates to a mostly-unuseful |undefined|. Code that relies on that is probably somewhat of an unusual case. Jeff _______________________________________________ dev-platform mailing list dev-platform@lists.mozilla.org https://lists.mozilla.org/listinfo/dev-platform
