On Monday, 2018-03-19 17:45:20 +0100, Guido Günther wrote: > Signed-off-by: Guido Günther <[email protected]>
It is indeed :) Reviewed-by: Eric Engestrom <[email protected]> > --- > clients/simple-dmabuf-drm.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/clients/simple-dmabuf-drm.c b/clients/simple-dmabuf-drm.c > index a184d7e7..22891639 100644 > --- a/clients/simple-dmabuf-drm.c > +++ b/clients/simple-dmabuf-drm.c > @@ -244,10 +244,10 @@ static int > fd_bo_export_to_prime(struct buffer *buf) > { > buf->dmabuf_fd = fd_bo_dmabuf(buf->fd_bo); > - if (buf->dmabuf_fd > 0) > - return 0; > + if (buf->dmabuf_fd < 0) > + return 1; > > - return 1; > + return 0; Nit: could be simplified as `return buf->dmabuf_fd < 0;` > } > > static int > -- > 2.16.1 > > _______________________________________________ > wayland-devel mailing list > [email protected] > https://lists.freedesktop.org/mailman/listinfo/wayland-devel _______________________________________________ wayland-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/wayland-devel
