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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to