ChuanqiXu added inline comments.

================
Comment at: clang/lib/Format/TokenAnnotator.cpp:3226
+    // No space between module :.
+    if (Left.isOneOf(Keywords.kw_module, tok::kw_export, Keywords.kw_import) &&
+        Right.is(TT_ModulePartitionColon))
----------------
owenpan wrote:
> You can remove `kw_export` as it must be followed by `import` or `module`, 
> based on how `TT_ModulePartitionColon` is set on Line 908.
`export` could be followed by many other declarations. (Maybe the logic on line 
908 is not right, I am not sure)


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

https://reviews.llvm.org/D114151

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

Reply via email to