[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Date: Wed, 4 May 2022 10:20:40 +0200
From: Jules Maselbas <jmaselbas@...ray.eu>
To: Patrick Oppenlander <patrick.oppenlander@...il.com>
Cc: musl@...ts.openwall.com
Subject: Re: [PATCH 1/2] tools: Add a script to remove trailing
spaces
Hi Patrick,
On Wed, May 04, 2022 at 04:06:44PM +1000, Patrick Oppenlander wrote:
> On Wed, May 4, 2022 at 1:57 AM Jules Maselbas <jmaselbas@...ray.eu> wrote:
> >
> > Add the remove-trailing-space.sh script, it will only consider files
> > tracked by git. Optionaly accept arguments to select the affected path.
> >
> > Signed-off-by: Jules Maselbas <jmaselbas@...ray.eu>
> > ---
> > tools/remove-trailing-space.sh | 2 ++
> > 1 file changed, 2 insertions(+)
> > create mode 100755 tools/remove-trailing-space.sh
> >
> > diff --git a/tools/remove-trailing-space.sh b/tools/remove-trailing-space.sh
> > new file mode 100755
> > index 00000000..a8b0056a
> > --- /dev/null
> > +++ b/tools/remove-trailing-space.sh
> > @@ -0,0 +1,2 @@
> > +#!/bin/sh
> > +git grep '[[:space:]]$' -- $@ | sed 's/:.*//' | uniq | xargs sed -i 's/[[:space:]]\+$//'
>
> There's a --name-only option you can give to git grep, so I think you
> can simplify this to:
Thanks, I've looked for this kind of option but failed to found it :)
> git grep --name-only '[[:space:]]$' | xargs sed -i 's/[[:space:]]\+$//'
This is much better :)
> Patrick
>
>
Powered by blists - more mailing lists
Confused about mailing lists and their use?
Read about mailing lists on Wikipedia
and check out these
guidelines on proper formatting of your messages.
