From: Ran Benita <ran234@gmail.com>
Date: Fri, 18 May 2012 16:39:25 +0000 (+0300)
Subject: rules: reformat LoadRules and XkbRF_Free
X-Git-Tag: accepted/2.0alpha-wayland/20121109.204519~64
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=7895eeb8ce4ebf5bf664530c6732f4c2fdf7abd7;p=profile%2Fivi%2Flibxkbcommon.git

rules: reformat LoadRules and XkbRF_Free

Signed-off-by: Ran Benita <ran234@gmail.com>
---

diff --git a/src/xkbcomp/rules.c b/src/xkbcomp/rules.c
index e6a5233..da36e68 100644
--- a/src/xkbcomp/rules.c
+++ b/src/xkbcomp/rules.c
@@ -1087,7 +1087,7 @@ add_group(struct rules *rules)
 }
 
 static struct rules *
-XkbcRF_LoadRules(FILE *file)
+load_rules(FILE *file)
 {
     struct input_line line;
     struct mapping mapping;
@@ -1107,36 +1107,39 @@ XkbcRF_LoadRules(FILE *file)
         if (match_line(&line, &mapping, &trule, &tgroup)) {
             if (tgroup.number) {
                 group = add_group(rules);
-	        if (group != NULL) {
-		    *group= tgroup;
-		    memset(&tgroup, 0, sizeof(tgroup));
-	        }
-	    } else {
+                if (group) {
+                    *group = tgroup;
+                    memset(&tgroup, 0, sizeof(tgroup));
+                }
+            } else {
                 rule = add_rule(rules);
-	        if (rule != NULL) {
-		    *rule= trule;
-		    memset(&trule, 0, sizeof(trule));
-	        }
-	    }
-	}
-	line.offset = 0;
+                if (rule) {
+                    *rule = trule;
+                    memset(&trule, 0, sizeof(trule));
+                }
+            }
+        }
+
+        line.offset = 0;
     }
+
     input_line_deinit(&line);
     return rules;
 }
 
 static void
-static void
-XkbcRF_Free(struct rules *rules)
+free_rules(struct rules *rules)
 {
     int i;
     struct rule *rule;
     struct group *group;
 
     if (!rules)
-	return;
+        return;
+
+    for (i = 0; i < rules->num_rules; i++) {
+        rule = &rules->rules[i];
 
-    for (i=0, rule = rules->rules; i < rules->num_rules && rules; i++, rule++) {
         free(rule->model);
         free(rule->layout);
         free(rule->variant);
@@ -1149,7 +1152,9 @@ XkbcRF_Free(struct rules *rules)
     }
     free(rules->rules);
 
-    for (i=0, group = rules->groups; i < rules->num_groups && group; i++, group++) {
+    for (i = 0; i < rules->num_groups; i++) {
+        group = &rules->groups[i];
+
         free(group->name);
         free(group->words);
     }
@@ -1185,7 +1190,7 @@ xkb_components_from_rules(struct xkb_context *ctx,
         return NULL;
     }
 
-    loaded = XkbcRF_LoadRules(rulesFile);
+    loaded = load_rules(rulesFile);
     if (!loaded) {
         ERROR("failed to load XKB rules \"%s\"\n", rulesPath);
         goto unwind_file;
@@ -1209,7 +1214,7 @@ xkb_components_from_rules(struct xkb_context *ctx,
     }
 
 unwind_file:
-    XkbcRF_Free(loaded);
+    free_rules(loaded);
     if (rulesFile)
         fclose(rulesFile);
     free(rulesPath);