[PATCH] D117416: [clang-format] Handle C variables with name that matches c++ access specifier

MyDeveloperDay via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 18 01:12:34 PST 2022


MyDeveloperDay added inline comments.


================
Comment at: .arclint:15-16
   }
-}
+}
\ No newline at end of file

----------------
curdeius wrote:
> Please don't change unrelated files.
I don't know about others but I personally don't use arcanist...

I stage my files, make a patch and upload manually

```
git add <files being modified>  (or `git ls files -m | xargs git add` )
git clang-format 
git diff --cached -U4 > patch_to_submitt.diff

```
I use the Create Diff/Update Diff functionality via the web interface (I feel that gives me more control)

{F21699407, size=full}

{F21699405, size=full}

Just my 2c


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117416



More information about the cfe-commits mailing list