Re: [PATCH] style: the opening '{' of a function is in a separate line

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Dec 10, 2018 at 7:47 PM Stefan Beller <sbeller@xxxxxxxxxx> wrote:
>
> On Sun, Dec 9, 2018 at 2:25 AM Nguyễn Thái Ngọc Duy <pclouds@xxxxxxxxx> wrote:
> >
> > Signed-off-by: Nguyễn Thái Ngọc Duy <pclouds@xxxxxxxxx>
> > ---
> >  bisect.c                   | 3 ++-
> >  builtin/config.c           | 3 ++-
> >  builtin/push.c             | 4 +++-
> >  convert.c                  | 3 ++-
> >  credential-cache--daemon.c | 3 ++-
> >  diff.c                     | 6 ++++--
> >  git.c                      | 3 ++-
> >  imap-send.c                | 3 ++-
> >  remote-curl.c              | 3 ++-
> >  sequencer.c                | 6 ++++--
> >  string-list.c              | 3 ++-
> >  t/helper/test-sigchain.c   | 3 ++-
> >  transport-helper.c         | 3 ++-
> >  url.c                      | 3 ++-
> >  userdiff.c                 | 3 ++-
> >  15 files changed, 35 insertions(+), 17 deletions(-)
>
> Can you say more about how you found these?

I was updating diff.c (65 patches coming soon!) and found one function
that triggered me. "git grep '^static.*) {$'" caught most of them. A
bit more regex got some more but I don't even know if I have caught
them all.

> (Are these all of them, or did you look through some subset of files?)

I left compat and xdiff directories out if I remember correctly.

> Or did you use a formatting tool?

Unfortunately no. Perhaps clang-format can do it? I didn't try.
-- 
Duy




[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux