On 1/6/2014, 8:05 PM, Karl Tomlinson wrote:
The bug is hidden almost as well by conversion of only indentation.if (condition1) if (condition2) foo(); else bar(); Sure, you could argue that style conversion makes the actual behaviour clearer, but you'd have to know the intended behaviour to know there was a bug. If we have a tool to skip the style change on any such unclear situations, then perhaps we can proceed more safely.
A syntactic change that's small, easy to review and limited to one part of one file, you say?
So as you describe it, a tool that backs off and flags ambiguities like this for human review would also be a tool that generates an _excellent_ set of Good First Bugs.
- mhoye _______________________________________________ dev-platform mailing list dev-platform@lists.mozilla.org https://lists.mozilla.org/listinfo/dev-platform
