Re: [PATCH] Btrfs: Coding style fixes

2016-12-13 Thread David Sterba
On Mon, Dec 12, 2016 at 07:28:46PM +0100, Seraphime Kirkovski wrote: > On Mon, Dec 12, 2016 at 05:11:56PM +0100, David Sterba wrote: > > This type of change is more like a cleanup and you can find more > > instances where the type is applied to just one of the operands, while > > min_t/max_t would

Re: [PATCH] Btrfs: Coding style fixes

2016-12-12 Thread Seraphime Kirkovski
On Mon, Dec 12, 2016 at 05:11:56PM +0100, David Sterba wrote: > This type of change is more like a cleanup and you can find more > instances where the type is applied to just one of the operands, while > min_t/max_t would be better. Feel free to send a separate patch for > that. Thanks for the

Re: [PATCH] Btrfs: Coding style fixes

2016-12-12 Thread David Sterba
On Mon, Dec 12, 2016 at 03:10:05PM +0100, Seraphime Kirkovski wrote: > Per Documentation/CodingStyle remove brackets on single expression > if statements, add missing spaces after `,` and around `=`, > remove unnecessary line continuations, add missing blank lines > after declarations and fix

[PATCH] Btrfs: Coding style fixes

2016-12-12 Thread Seraphime Kirkovski
Per Documentation/CodingStyle remove brackets on single expression if statements, add missing spaces after `,` and around `=`, remove unnecessary line continuations, add missing blank lines after declarations and fix mixed spaces and tabs. Signed-off-by: Seraphime Kirkovski