On 8 February 2017 at 13:42, Pekka Paalanen <[email protected]> wrote:
> On Tue,  7 Feb 2017 17:29:53 -0500
> Micah Fedke <[email protected]> wrote:
>
Where is the commit message ? Here is what comes to mind here, feel
free to reuse.

"As of Wayland vX the $NEW_API was introduced to replace the
deprecated $STRUCT_FOO. Seemingly due to a unrelated bug $REFERENCE
none of the deprecation warning triggered."

>
> Hi,
>
> this looks good to me:
> Reviewed-by: Pekka Paalanen <[email protected]>
>
> It would also be good to add #define WL_HIDE_DEPRECATED to all
> wayland-related .c files before #includes so that the compiler will
> catch any attempts to use the deprecated definitions again.
>
Fully agree, but let's keep it follow-up patch. I'm split if we should
#define in each .c files or once in build system.
The latter sounds shorter + more robust.

Thanks
Emil
_______________________________________________
mesa-dev mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to