[PATCH] D115967: [clang-format][NFC] Handle wrapping after => in mustBreakBefore()

[Peter Stys via Phabricator via cfe-commits]

peterstys added inline comments.


================
Comment at: clang/lib/Format/UnwrappedLineParser.cpp:1955-1957
+    if (Style.isCSharp() && FormatTok->is(TT_FatArrow) &&
+        tryToParseChildBlock())
+      continue;
----------------
owenpan wrote:
> peterstys wrote:
> > It seems that this block could be removed altogether, because this is already done on lines 1964-165, and it's done regardless if it the language is C# or not.
> Lines 1964-1965 only apply to JavaScript because of line 1958, so lines 1955-1957 cannot be removed.
Oh yes, sorry I missed the Javascript block. I wonder if in that case wouldn't be more consistent to add Javascript check on the line 1955 to be consistent with how the same block of logic is handled on the lines 1641-1645.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D115967/new/

https://reviews.llvm.org/D115967