[clang] [clang-format] Add null-terminated path option (#123921) (PR #123926)
via cfe-commits
cfe-commits at lists.llvm.org
Fri Jan 24 02:53:34 PST 2025
mydeveloperday wrote:
> > This seems reasonable to me, but wait for the others, maybe @owenca
>
> I'm wondering if `-0` should turn off all informational messaging as well and _only_ print the paths. As it is, it will print things like _Running clang-format on the following files:_, even if a null-separated list of files follows.
>
> So right now the usage would be (example)
>
> ```shell
> git clang-format --staged -0 | tail -n +2 | xargs -0 git add
> ```
>
> but perhaps it should not print informational messages and have the usage be simplified:
>
> ```shell
> git clang-format --staged -0 | xargs -0 git add
> ```
Aren't you really asking, just give me a list of all the files that have been changed by clang format, in which case -0 isn't really that descriptive, what not
```
--modified or -m (similar to git ls-files)
```
or
``
-- affected (might avoid becausee of affected/effected spelling issues people have)
``
https://github.com/llvm/llvm-project/pull/123926
More information about the cfe-commits
mailing list