> He didn't mean to reduce the length of the line to 72, but to align
> the number (328 bytes) in the example with the last column of previous
> lines. Is one thing I also noted that your patch changed. Note the
> final 'bytes' word changed side.
Ohh okay now i see! Thank you Xabier and Dietmar
Hi Jones,
> > How about
> >
> > 72 (add/remove: 3/0 grow/shrink: 2/0 up/down: 328/0) Total bytes:
> > 328
> >
> > to keep the numbers aligned?
>
> Yes! 72 is totally aligned with function/old/new/delta length! :)
> And IMHO 75 columns is still a bit preferred than 72 columns, two reasons
> How about
>
> 72 (add/remove: 3/0 grow/shrink: 2/0 up/down: 328/0) Total bytes: 328
>
> to keep the numbers aligned?
Yes! 72 is totally aligned with function/old/new/delta length! :)
And IMHO 75 columns is still a bit preferred than 72 columns, two reasons
are considered:
1.
> From: busybox On Behalf Of Jones Syue ???
> Sent: Wednesday, March 13, 2024 2:22 AM
>
> > This patch replaces the 78 "-" prints with 75 "-". And replace the 80
> > columns summary line with 77 columns. ("%s" is considered as two chars
> > and should be filled with whitespace " ", so 77 = 75 +