I think it is, it makes the behavior consistent independent of the
superfluous semicolon at the end. I can add a test case in one of my next
commits.


On Wed, May 7, 2014 at 1:50 PM, Tobias Grosser <[email protected]> wrote:

> On 07/05/2014 13:20, Daniel Jasper wrote:
>
>> Why is this using an elaborated return type? Can you just remove the
>> "struct"?
>>
>
> I just rebuilt clang-format and I found another formatting change:
>
> -  ReportFuncCall(Instruction *Inst) : Inst(Inst) {
> -    ++BadFuncCallForScop;
> -  };
> +  ReportFuncCall(Instruction *Inst) : Inst(Inst) { ++BadFuncCallForScop;
> }; <- This is just one line
>
> Was this intentional as well? In case it was, maybe it is worth adding
> some non-js tests here.
>
> Tobias
>
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to