It's ok like this. But I think it would be nice to move this function inside the existing #ifdef HAVE_WAYLAND_BACKEND ... #endif, where it is used, to keep all related wayland code close.
Jose ----- Original Message ----- > Fixes this GCC warning. > native_drm.c:153:1: warning: ‘drm_display_authenticate’ defined but > not > used [-Wunused-function] > > Signed-off-by: Vinson Lee <[email protected]> > --- > src/gallium/state_trackers/egl/drm/native_drm.c | 2 ++ > 1 files changed, 2 insertions(+), 0 deletions(-) > > diff --git a/src/gallium/state_trackers/egl/drm/native_drm.c > b/src/gallium/state_trackers/egl/drm/native_drm.c > index c013769..3652581 100644 > --- a/src/gallium/state_trackers/egl/drm/native_drm.c > +++ b/src/gallium/state_trackers/egl/drm/native_drm.c > @@ -149,6 +149,7 @@ static struct native_display_buffer > drm_display_buffer = { > drm_display_export_native_buffer > }; > > +#ifdef HAVE_WAYLAND_BACKEND > static int > drm_display_authenticate(void *user_data, uint32_t magic) > { > @@ -157,6 +158,7 @@ drm_display_authenticate(void *user_data, > uint32_t magic) > > return drmAuthMagic(drmdpy->fd, magic); > } > +#endif > > static char * > drm_get_device_name(int fd) > -- > 1.7.9 > > _______________________________________________ > mesa-dev mailing list > [email protected] > http://lists.freedesktop.org/mailman/listinfo/mesa-dev > _______________________________________________ mesa-dev mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/mesa-dev
