owenpan added a comment.

Do you need to update the current diff to show the net change? For example, the 
unit tests you added don't show up as a diff right now.



================
Comment at: clang/include/clang/Format/Format.h:95-98
     /// \warning
-    ///  Note: This currently only applies to parentheses.
+    ///  Note: This currently only applies to parentheses and braced list
+    ///  initializers when ``Cpp11BracedListStyle`` is ``True``.
     /// \endwarning
----------------
gedare wrote:
> owenpan wrote:
> > 
> This `\note` is not currently supported? I have submitted a rev D154552 to 
> add such ability.
Ah, you are right. Please still use `warning` here and update D154552 after 
this patch lands.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153205

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to