[PATCH] D101702: [clang-format] Add more support for C# 8 nullables

Eliza via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue May 4 11:53:51 PDT 2021


exv marked an inline comment as done.
exv added a comment.

According to the wiki, it seems like someone who has commit access must now submit this change.



================
Comment at: clang/lib/Format/FormatTokenLexer.cpp:102
+
+    // FIXME: Investigate what token type gives the correct operator priority.
+    if (tryMergeTokens(FatArrow, TT_JsFatArrow))
----------------
curdeius wrote:
> Could you explain in what case the operator precedence may be wrong here?
It's unlikely this comment is still relevant for this section. I will delete it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101702



More information about the cfe-commits mailing list