st/dri: add note about GL interop version checks
authorEmil Velikov <emil.velikov@collabora.com>
Tue, 3 May 2016 10:13:12 +0000 (11:13 +0100)
committerEmil Velikov <emil.l.velikov@gmail.com>
Tue, 24 May 2016 22:03:00 +0000 (23:03 +0100)
... and make them more explicit.

Signed-off-by: Emil Velikov <emil.velikov@collabora.com>
Reviewed-by: Marek Olšák <marek.olsak@amd.com>
Tested-by: Tom Stellard <thomas.stellard@amd.com>
src/gallium/state_trackers/dri/dri2.c

index 8b6af600e3221c6a1804839b206ae3a6105a108a..15e34431881ebcb8e2afa8218f1345c1abdd46a4 100644 (file)
@@ -1468,7 +1468,8 @@ dri2_interop_query_device_info(__DRIcontext *_ctx,
 {
    struct pipe_screen *screen = dri_context(_ctx)->st->pipe->screen;
 
-   if (!out->version)
+   /* There is no version 0, thus we do not support it */
+   if (out->version == 0)
       return MESA_GLINTEROP_INVALID_VERSION;
 
    out->pci_segment_group = screen->get_param(screen, PIPE_CAP_PCI_GROUP);
@@ -1497,7 +1498,8 @@ dri2_interop_export_object(__DRIcontext *_ctx,
    unsigned target, usage;
    boolean success;
 
-   if (!in->version || !out->version)
+   /* There is no version 0, thus we do not support it */
+   if (in->version == 0 || out->version == 0)
       return MESA_GLINTEROP_INVALID_VERSION;
 
    /* Validate the target. */