Re: [PATCH] D24574: clang-format: [JS] ASI insertion after boolean literals.

2016-09-18 Thread Martin Probst via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL281856: clang-format: [JS] ASI insertion after boolean literals. (authored by mprobst). Changed prior to commit: https://reviews.llvm.org/D24574?vs=71385&id=71755#toc Repository: rL LLVM https://rev

Re: [PATCH] D24574: clang-format: [JS] ASI insertion after boolean literals.

2016-09-14 Thread Daniel Jasper via cfe-commits
djasper accepted this revision. djasper added a comment. This revision is now accepted and ready to land. Interesting.. Would have expected true/false to be literals as well, but ok :). https://reviews.llvm.org/D24574 ___ cfe-commits mailing list cf

[PATCH] D24574: clang-format: [JS] ASI insertion after boolean literals.

2016-09-14 Thread Martin Probst via cfe-commits
mprobst created this revision. mprobst added a reviewer: djasper. mprobst added a subscriber: cfe-commits. Herald added a subscriber: klimek. Before when a semicolon was missing after a boolean literal: a = true return 1; clang-format would parse this as one line and format as: a = tr