commit 7eb54bacadadc746cd614fde1c6abaa04ddc45d0
author Radek Krejci <rkrejci@cesnet.cz> Mon May 18 16:30:04 2020 +0200
committer Radek Krejci <rkrejci@cesnet.cz> Mon May 18 16:42:47 2020 +0200
tree 9501f212308475acdd24cfc3fe44be7b70d45f09
parent d91d4da606bdf094204a2b7b2ee7d3d8f81e9321

libyang CHANGE use LY_ARRAY_SIZE_TYPE for iterator variables in LY_TREE_FOR

Because of the change in sized arrays design (780ade2101), also the
iterators and other variables used with LY_TREE_FOR or LY_TREE_SIZE
should be of LY_ARRAY_SIZE_TYPE.

Also the new LY_PRI_ARRAY_SIZE_TYPE was introduced to correctly print
the value of this type in a formatted print functions.

src/tree_schema.c

diff --git a/src/tree_schema.c b/src/tree_schema.c
index 176664a..9a2ee1f 100644
--- a/src/tree_schema.c
+++ b/src/tree_schema.c
@@ -42,7 +42,7 @@
     int action_flag = 0, notif_flag = 0;
     const struct lysc_action *actions;
     const struct lysc_notif *notifs;
-    unsigned int u;
+    LY_ARRAY_SIZE_TYPE u;
 
     LY_CHECK_ARG_RET(NULL, parent || module, NULL);
 
@@ -374,7 +374,8 @@
 lys_feature_change(const struct lys_module *mod, const char *name, int value)
 {
     int all = 0;
-    unsigned int u, changed_count, disabled_count;
+    LY_ARRAY_SIZE_TYPE u, disabled_count;
+    uint32_t changed_count;
     struct lysc_feature *f, **df;
     struct lysc_iffeature *iff;
     struct ly_set *changed;
@@ -535,7 +536,7 @@
 {
     struct lysc_feature *f;
     struct lysc_module *mod;
-    unsigned int u;
+    LY_ARRAY_SIZE_TYPE u;
 
     LY_CHECK_ARG_RET(NULL, module, module->compiled, feature, -1);
     mod = module->compiled;
@@ -559,7 +560,7 @@
 API const struct lysc_node *
 lysc_node_is_disabled(const struct lysc_node *node, int recursive)
 {
-    unsigned int u;
+    LY_ARRAY_SIZE_TYPE u;
 
     LY_CHECK_ARG_RET(NULL, node, NULL);
 
@@ -713,7 +714,7 @@
     struct lysp_import *imp;
     struct lysp_include *inc;
     LY_ERR ret = LY_EINVAL;
-    unsigned int u, i;
+    LY_ARRAY_SIZE_TYPE u, v;
     struct lys_yang_parser_ctx *yangctx = NULL;
     struct lys_yin_parser_ctx *yinctx = NULL;
     struct lys_parser_ctx *pctx;
@@ -824,8 +825,8 @@
             goto error_ctx;
         }
         /* check for importing the same module twice */
-        for (i = 0; i < u; ++i) {
-            if (imp->module == mod->parsed->imports[i].module) {
+        for (v = 0; v < u; ++v) {
+            if (imp->module == mod->parsed->imports[v].module) {
                 LOGVAL(ctx, LY_VLOG_NONE, NULL, LYVE_REFERENCE, "Single revision of the module \"%s\" referred twice.", imp->name);
                 goto error_ctx;
             }