On 31 March 2017 at 11:04, Eric Blake <ebl...@redhat.com> wrote: > > Might also be worth adding that an audit of the code finds no semantic > change, that this is just cleaning up the compiler warning.
We should include core detail of your detailed analysis in such a statement I think, because I it's more than just cleaning up a warning. For me at least the warning pointed out that the expression is not parsed would be expected (if this were a function and not a macro), and it's just because of constraints on the inputs that there's no functional change. >> Signed-off-by: Corey Minyard <cminy...@mvista.com> > Reviewed-by: Eric Blake <ebl...@redhat.com> Reviewed-by: Ed Maste <ema...@freebsd.org>