On Fri, Aug 25, 2017 at 1:20 PM, Emil Velikov <[email protected]>
wrote:

> From: Emil Velikov <[email protected]>
>
> Originally required for the u_vector implementation, which was inlined
> in u_vector.h with previous commit.
>
> Using libmesautil pulled the C++ runtime (string_to_uint_map.cpp),
> which is something don't want to impose in our libEGL.
>
> We could consider rewriting string_to_uint_map in C, but that's too
> invasive for a stable fix.
>

A quick grep and it looks like the only users of string_to_uint_map are
src/compiler/glsl and src/mesa which depends on src/compiler/glsl.  Why not
just move string_to_uint_map into src/compiler/glsl or src/compiler until
such a time as we actually have another user.  Don't get me wrong, I think
string_to_uint_map is useful but I think the better option here is to
disallow C++ in src/util.

--Jason


> Cc: Daniel Stone <[email protected]>
> Cc: "17.2" <[email protected]>
> Fixes: 02cc35937277 ("egl/wayland: Use linux-dmabuf interface for buffers")
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=101851
> Signed-off-by: Emil Velikov <[email protected]>
> ---
>  src/egl/Makefile.am | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/src/egl/Makefile.am b/src/egl/Makefile.am
> index bb8ec9745dd..7331b616a8d 100644
> --- a/src/egl/Makefile.am
> +++ b/src/egl/Makefile.am
> @@ -82,7 +82,6 @@ AM_CFLAGS += $(WAYLAND_CFLAGS)
>  libEGL_common_la_LIBADD += $(WAYLAND_LIBS)
>  libEGL_common_la_LIBADD += $(LIBDRM_LIBS)
>  libEGL_common_la_LIBADD += $(top_builddir)/src/egl/wayland/wayland-drm/
> libwayland-drm.la
> -libEGL_common_la_LIBADD += $(top_builddir)/src/util/libmesautil.la
>  dri2_backend_FILES += \
>         drivers/dri2/platform_wayland.c \
>         drivers/dri2/linux-dmabuf-unstable-v1-protocol.c \
> --
> 2.13.3
>
> _______________________________________________
> mesa-dev mailing list
> [email protected]
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
_______________________________________________
mesa-dev mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to