[PATCH] D24257: clang-format: [JS] ignore comments when wrapping returns.
Martin Probst via cfe-commits
cfe-commits at lists.llvm.org
Tue Sep 6 08:25:17 PDT 2016
mprobst added inline comments.
================
Comment at: lib/Format/TokenAnnotator.cpp:2384
@@ -2383,2 +2383,3 @@
} else if (Style.Language == FormatStyle::LK_JavaScript) {
- if (Left.is(tok::kw_return))
+ const FormatToken *NonComment = Right.getPreviousNonComment();
+ if (Left.isOneOf(tok::kw_return, tok::kw_continue, tok::kw_break,
----------------
Should the entire method generally consider the previous non-comment, instead of just the previous token? Not sure if that makes sense in all cases, and I don't think we always have a previous non-comment token either.
https://reviews.llvm.org/D24257
More information about the cfe-commits
mailing list