glx: change query_renderer_integer() value param to unsigned
When this function was added, the returned value was signed in some places, unsigned in others. v2: also add unsigned in the unit test, per Ian. Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
This commit is contained in:
+1
-1
@@ -477,7 +477,7 @@ struct glx_screen_vtable {
|
|||||||
unsigned *error);
|
unsigned *error);
|
||||||
int (*query_renderer_integer)(struct glx_screen *psc,
|
int (*query_renderer_integer)(struct glx_screen *psc,
|
||||||
int attribute,
|
int attribute,
|
||||||
int *value);
|
unsigned int *value);
|
||||||
int (*query_renderer_string)(struct glx_screen *psc,
|
int (*query_renderer_string)(struct glx_screen *psc,
|
||||||
int attribute,
|
int attribute,
|
||||||
const char **value);
|
const char **value);
|
||||||
|
|||||||
@@ -30,7 +30,7 @@ __glXQueryRendererInteger(struct glx_screen *psc, int attribute,
|
|||||||
unsigned int *value)
|
unsigned int *value)
|
||||||
{
|
{
|
||||||
unsigned int values_for_query = 0;
|
unsigned int values_for_query = 0;
|
||||||
int buffer[32];
|
unsigned int buffer[32];
|
||||||
int err;
|
int err;
|
||||||
|
|
||||||
/* This probably means the caller is trying to use an extension function
|
/* This probably means the caller is trying to use an extension function
|
||||||
|
|||||||
@@ -56,7 +56,8 @@ static bool query_renderer_string_called = false;
|
|||||||
static bool query_renderer_integer_called = false;
|
static bool query_renderer_integer_called = false;
|
||||||
|
|
||||||
static int
|
static int
|
||||||
fake_query_renderer_integer(struct glx_screen *psc, int attribute, int *value)
|
fake_query_renderer_integer(struct glx_screen *psc, int attribute,
|
||||||
|
unsigned int *value)
|
||||||
{
|
{
|
||||||
(void) psc;
|
(void) psc;
|
||||||
(void) attribute;
|
(void) attribute;
|
||||||
|
|||||||
Reference in New Issue
Block a user