Re: [PATCH] 2.4.5pre3 warning fixes

From: Albert D. Cahalan (acahalan@cs.uml.edu)
Date: Thu May 17 2001 - 23:44:57 EST


Bingner Sam J. Con writes:

> Looks to me like it's adding { and } on each side of the
> "c->devices->prev=d;" statement... so changing from:
>
> if (c->devices != NULL)
> c->devices->prev=d;
>
> to
>
> if (c->devices != NULL){
> c->devices->prev=d;
> }
>
> I assume the new compiler likes the if to have explicit
> brackets instead of using the next statement...

Maybe one of these will make it happy:

(void)(c->devices && (c->devices->prev=d));

!c->devices ?: (c->devices->prev=d);
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Wed May 23 2001 - 21:00:26 EST