On 07/25/2011 06:38 PM, malc wrote:
On Mon, 25 Jul 2011, Avi Kivity wrote:
> On 07/25/2011 06:22 PM, malc wrote:
> > > > > + }
> > > > > + if (a.fd< b.fd) {
> > > > > + return true;
> > > > > + } else {
> > if (a.fd> b.fd) {
> > > > > + return false;
> > > > > + }
> > }
>
> CODING_STYLE permits braceless else if.
The example uses it, the text however suggests it's illegal.
I'll post a patch. It's perfectly reasonable IMO, and there ~1800
instances in the tree.
--
error compiling committee.c: too many arguments to function
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
