krasimir added inline comments.

================
Comment at: clang/unittests/Format/FormatTestJS.cpp:194
-            format("/** @returns {string} jsdoc line 12 */",
-                   getGoogleJSStyleWithColumns(20)));
   EXPECT_EQ("/**\n"
----------------
this actually demonstrates a bug fixed by this patch: with 20 column limit, the 
comment should be split. The new behavior makes the test with a column limit of 
25 redundant, so folded the two of them.


================
Comment at: clang/unittests/Format/FormatTestJS.cpp:205
-            " */",
-            format("/** @returns {string} jsdoc line 12  */",
                    getGoogleJSStyleWithColumns(20)));
----------------
this is an exact duplicate as the check on old line 190; removed


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91078

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

Reply via email to