On Wed, Mar 09, 2011 at 03:40:18PM +0100, Henrik Rydberg wrote: > On Wed, Mar 09, 2011 at 02:28:13PM +0000, Mark Brown wrote: > > On Wed, Mar 09, 2011 at 03:11:39PM +0100, Henrik Rydberg wrote: > > > > > > - wm831x_ts->pressure = pdata && pdata->pressure; > > > > + if (pdata) > > > > + wm831x_ts->pressure = pdata->pressure; > > > > + else > > > > + wm831x_ts->pressure = true; > > > > > wm831x_ts->pressure = !pdata || pdata->pressure; > > > > > should suffice. > > > > It does have some substantial drawbacks in terms of legibility, though. > > I suppose that is in the eye of the beholder. > Applied, thanks Mark (even though I also prefer more concise variants). -- Dmitry -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
