Re: [PATCH] sched_ext: Add missing braces to multi-line blocks

[Cheng-Yang Chou <yphbchou0911@gmail.com>]

On Wed, Mar 04, 2026 at 07:50:31AM -1000, Tejun Heo wrote:
> Hello,
> 
> On Thu, Mar 05, 2026 at 01:12:59AM +0800, Cheng-Yang Chou wrote:
> > On Mon, Mar 02, 2026 at 03:13:32PM +0800, Cheng-Yang Chou wrote:
> > > Add braces around outer loops and conditionals that contain multi-line
> > > inner statements in both ext.c and ext_idle.c.
> > > 
> > > According to the kernel maintainer tip bracket rules, brackets should
> > > only be omitted if the statement following an 'if' or 'for' is truly
> > > a single line. This improves reading flow and prevents future errors.
> > > 
> > > Link: https://www.kernel.org/doc/html/latest/process/maintainer-tip.html#bracket-rules
> > > 
> > 
> > Gentle ping on this minor style fix.
> > No rush at all, just making sure it didn't slip by.
> 
> I don't think these trivial patches are useful. Ask yourself, is the code
> meaningfully better afterwards? Maybe as a part of some other big changes,
> these might make sense, but, by themselves, it's just noise, a busy work.
> Please don't send these patches.
> 
> Thanks.
> 
> -- 
> tejun

Understood.
Thanks for the feedback!

-- 
Thanks,
Cheng-Yang