Re: [PATCH v2 bpf-next] bpftool: introduce btf c dump sorting

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

 



On 10/05/2024 00:24, Alexei Starovoitov wrote:
> On Thu, May 9, 2024 at 4:09 PM Quentin Monnet <qmo@xxxxxxxxxx> wrote:
>>
>> Nit: Most variables in the file are declared in "reverse-Christmas-tree"
>> order (longest lines first, unless there's a reason not to). Could you
>> please try to preserve this order, here and elsewhere, for consistency?
> 
> I so hate this nitpick.

Yes, I've seen you push back before; I was wondering if you'd react on
this one. I don't mind too much about the ordering, to be honest. To me,
this is mostly a matter of consistency in the file.

> I'll start introducing non-rev-xmas tree everywhere just
> to stop this madness.

That's maybe not necessary :), at least as far as I'm concerned. I'm
fine with dropping this for future reviews (starting with the current one).

Quentin




[Index of Archives]     [Linux Samsung SoC]     [Linux Rockchip SoC]     [Linux Actions SoC]     [Linux for Synopsys ARC Processors]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]


  Powered by Linux