[Dnsmasq-discuss] space two point eighty one, take 2

Geert Stappers stappers at stappers.nl
Sat Mar 7 21:01:30 GMT 2020


In-Reply-To: <1255b7ff-351f-46d7-f811-ac3373102662 at thekelleys.org.uk>
On Thu, Mar 05, 2020 at 10:34:29PM +0000, Simon Kelley wrote:
> On 05/03/2020 21:05, Geert Stappers wrote:
> > On Thu, Mar 05, 2020 at 09:46:21PM +0100, Geert Stappers wrote:
> >>
> >> I plea for removal of unneeded ' ' and '	'.
> >>
> >> That removal can be done with:
> >>   ... remove trailing white_space_only_lines ...
> >>
> >>   ... empty lines: sed --in-place -e 's/^[ \t]*$//' src/*.c   ...
> > 
> > I love to see that in the 2.81 release of dnsmasq.
> > 
> 
> The obvious problem with doing that is that for ever more, when I run
} "git blame" N lines will have the source "Removed useless whitespace".

That valid point shouldn't be a big issue.
At least I think it can be mitigated. 

Now spliting previous change request into two parts:
* Removing trailing white_space_only_lines
* Emptying the empty lines


To be continued.


Regards
Geert Stappers
-- 
Silence is hard to parse



More information about the Dnsmasq-discuss mailing list