On Tue, Mar 12, 2019 at 09:43:39PM +0200, Leon Romanovsky wrote: > On Tue, Mar 12, 2019 at 12:36:48PM -0700, Bart Van Assche wrote: > > On Mon, 2019-03-11 at 14:33 +0200, Leon Romanovsky wrote: > > > - _s >= 0 && _s < 8 * sizeof(*d) ? _s : 0; \ > > > + (_s == 0 || _s > 0) && _s < 8 * sizeof(*d) ? _s : 0; \ > > > > Hi Leon, > > > > Is there any particular reason why (_s == 0 || _s > 0) is used above instead > > of is_non_negative(_s)? > > There is no reason, except that I wanted to get rid of this fix as soon > as possible, this warning distract users of my CI system. So v3 then? Jason