14 Jul
2021
14 Jul
'21
9:04 a.m.
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@redhat.com> ---
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.