owenpan added inline comments.

================
Comment at: clang/lib/Format/TokenAnnotator.cpp:3219
+  // of comment.
+  if (Left.is(tok::star) && Right.is(TT_BlockComment))
+    return true;
----------------
MyDeveloperDay wrote:
> owenpan wrote:
> > Isn't `tok::comment` better than `TT_BlockComment` if a space is also 
> > required between `*` and `//`?
> I guess for formatting as we type, then perhaps that might be better, I just 
> couldn't think of a use case for legal code where * would be the last value 
> on the line, but perhaps
> 
> ```
> void foo(int *  // this is the first parameter
>               ,int second);
> ```
> 
> Let me add that as a test and update the review.
Maybe something like:
```
  double term = factor1 * // first factor
                factor2;
```


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114142

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

Reply via email to