On Thu, Jan 19, 2017 at 08:21:45AM +1000, Peter Hutterer wrote: > clients/editor.c: In function ‘read_file’: > clients/editor.c:1578:16: warning: logical ‘or’ applied to non-boolean > constant [-Wlogical-op] > errno = errsv || EINVAL; > > This works in the shell, but not in C. Introduced in 411ffabbb56b > > Signed-off-by: Peter Hutterer <peter.hutte...@who-t.net>
LGTM, Reviewed-by: Bryce Harrington <br...@osg.samsung.com> > --- > Changes to v1: > - first version didn't return the right errno, whoops :) > > clients/editor.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/clients/editor.c b/clients/editor.c > index f1dffe1..a0cc97a 100644 > --- a/clients/editor.c > +++ b/clients/editor.c > @@ -1575,7 +1575,7 @@ error: > if (fin) > fclose(fin); > free(buffer); > - errno = errsv || EINVAL; > + errno = errsv ? errsv : EINVAL; > > return NULL; > } > -- > 2.9.3 > > _______________________________________________ > wayland-devel mailing list > wayland-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/wayland-devel _______________________________________________ wayland-devel mailing list wayland-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-devel