Messages in this thread | | | Date | Thu, 31 Mar 2022 19:34:11 -0400 | From | Steven Rostedt <> | Subject | Re: [PATCH] ftrace: remove check of list iterator against head past the loop body |
| |
On Fri, 1 Apr 2022 01:22:58 +0200 Jakob Koschel <jakobkoschel@gmail.com> wrote:
> > Can you move this to the first declaration to keep the nice "upside-down > > x-mas tree" look. > > Thanks, I'll fix that up. It seems like this is not applied on the entire kernel > making treewide changes a bit more difficult. Is it documented somewhere, which > parts of the kernel enforce this? Just looking two lines down from here it > seems to be 'broken' already so just from looking at existing code it's often > hard to judge.
It's one of those things that some maintainers prefer (I'm one of them ;-) because it makes it easier to read IMHO.
But as you noticed, it's broken even in the same file. That's because I don't strictly enforce it. If there's a lot of code that looks good to go in, I don't ask to fix it. But as this was a small trivial patch, I figured I'd mention it.
Thus, it's something that you do when asked, but don't worry about doing it across the board, you are not going to upset anybody by forgetting to do it.
-- Steve
| |