gbm: use size_t for array indexes
authorEric Engestrom <eric@engestrom.ch>
Sun, 1 Sep 2019 16:42:23 +0000 (17:42 +0100)
committerEric Engestrom <eric.engestrom@intel.com>
Sun, 13 Oct 2019 17:10:47 +0000 (18:10 +0100)
Signed-off-by: Eric Engestrom <eric@engestrom.ch>
Reviewed-by: Eric Anholt <eric@anholt.net>
src/gbm/backends/dri/gbm_dri.c

index ce24c3183e02054692b52c5ca67bf2a8953d8a5c..8fe6a0265fbfc5e32fe15c9a47ce561cc5d4dadf 100644 (file)
@@ -281,12 +281,11 @@ dri_bind_extensions(struct gbm_dri_device *dri,
                     struct dri_extension_match *matches, size_t num_matches,
                     const __DRIextension **extensions)
 {
-   int i, j;
    bool ret = true;
    void *field;
 
-   for (i = 0; extensions[i]; i++) {
-      for (j = 0; j < num_matches; j++) {
+   for (size_t i = 0; extensions[i]; i++) {
+      for (size_t j = 0; j < num_matches; j++) {
          if (strcmp(extensions[i]->name, matches[j].name) == 0 &&
              extensions[i]->version >= matches[j].version) {
             field = ((char *) dri + matches[j].offset);
@@ -295,7 +294,7 @@ dri_bind_extensions(struct gbm_dri_device *dri,
       }
    }
 
-   for (j = 0; j < num_matches; j++) {
+   for (size_t j = 0; j < num_matches; j++) {
       field = ((char *) dri + matches[j].offset);
       if ((*(const __DRIextension **) field == NULL) && !matches[j].optional) {
          ret = false;
@@ -571,10 +570,8 @@ static const struct gbm_dri_visual gbm_dri_visuals_table[] = {
 static int
 gbm_format_to_dri_format(uint32_t gbm_format)
 {
-   int i;
-
    gbm_format = gbm_format_canonicalize(gbm_format);
-   for (i = 0; i < ARRAY_SIZE(gbm_dri_visuals_table); i++) {
+   for (size_t i = 0; i < ARRAY_SIZE(gbm_dri_visuals_table); i++) {
       if (gbm_dri_visuals_table[i].gbm_format == gbm_format)
          return gbm_dri_visuals_table[i].dri_image_format;
    }
@@ -585,9 +582,7 @@ gbm_format_to_dri_format(uint32_t gbm_format)
 static uint32_t
 gbm_dri_to_gbm_format(int dri_format)
 {
-   int i;
-
-   for (i = 0; i < ARRAY_SIZE(gbm_dri_visuals_table); i++) {
+   for (size_t i = 0; i < ARRAY_SIZE(gbm_dri_visuals_table); i++) {
       if (gbm_dri_visuals_table[i].dri_image_format == dri_format)
          return gbm_dri_visuals_table[i].gbm_format;
    }