Re: [PATCH] Use postfix increment

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

 



On Tue, Jul 13, 2021 at 14:18:24 +0200, Michal Prívozník wrote:
> On 7/13/21 11:11 AM, Peter Krempa wrote:
> > On Tue, Jul 13, 2021 at 08:44:06 +0200, Michal Privoznik wrote:
> >> We document that our coding style is to use postfix increment in
> >> for() loops rather than prefix. This change was generated by the
> >> following sed script:
> >>
> >>   for i in $(git grep -l "for (.*; ++.*)"); do \
> >>     sed -i 's/\(for (.*; \)++\(.*\))/\1\2++)/' $i; \
> >>   done
> >>
> >> Signed-off-by: Michal Privoznik <mprivozn@xxxxxxxxxx>
> >> ---
> > 
> > NACK this is mostly pointless churn.
> > 
> 
> Fair enough. I just wanted us to be compliant with our coding style
> guide. But I can live with this patch unmerged.

I'll be okay if you add a syntax check, but just fixing it is not worth
it in this instance.




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux