Re: [PATCH v4] fs/aio.c: Cosmetic

2020-11-21 Thread Alejandro Colomar
Hi Randy, Thanks for the review. Next time I'll do it in two parts, as you wished. Thanks, Alex On 11/21/20 2:22 AM, Randy Dunlap wrote: > On 11/20/20 2:06 PM, Alejandro Colomar wrote: >> Changes: >> - Consistently use 'unsigned int', instead of 'unsigned'. >> - Add a blank line after variable

Re: [PATCH v4] fs/aio.c: Cosmetic

2020-11-20 Thread Randy Dunlap
On 11/20/20 2:06 PM, Alejandro Colomar wrote: > Changes: > - Consistently use 'unsigned int', instead of 'unsigned'. > - Add a blank line after variable declarations. > - Move variable declarations to the top of functions. > - Invert logic of 'if's to reduce indentation and simplify function logic.

[PATCH v4] fs/aio.c: Cosmetic

2020-11-20 Thread Alejandro Colomar
Changes: - Consistently use 'unsigned int', instead of 'unsigned'. - Add a blank line after variable declarations. - Move variable declarations to the top of functions. - Invert logic of 'if's to reduce indentation and simplify function logic. - Add goto tags when needed. - Early re