Re: [PATCH 1/3] KVM: x86 emulator: Fix task switch privilege checks

[Gleb Natapov <gleb@redhat.com>]

On Tue, Jan 24, 2012 at 11:09:09AM +0100, Kevin Wolf wrote:
> >> +	} else if (reason != TASK_SWITCH_IRET) {
> >> +		dpl = next_tss_desc.dpl;
> >>  	}
> > No need parentheses around one statement.
> 
> Documentation/CodingStyle says:
> 
> "This does not apply if only one branch of a conditional statement is a
> single statement; in the latter case use braces in both branches:"
> 
Then you need to put parentheses around "if (reason != TASK_SWITCH_IRET)"
if you want to follow the letter of the CodingStyle :)
But I do not see this coding stile part widely used in core kernel code:
$ git grep "} else$" kernel | wc -l
122

Can't think of re to check when the rule is followed :(

--
			Gleb.