Browse Source

Add angle brackets to a single option/flag rule (#1399)

Huidae Cho 4 years ago
parent
commit
d2dd0dd262
1 changed files with 6 additions and 4 deletions
  1. 6 4
      lib/gis/parser_dependencies.c

+ 6 - 4
lib/gis/parser_dependencies.c

@@ -160,14 +160,16 @@ static int count_present(const struct rule *rule, int start)
 static const char *describe_rule(const struct rule *rule, int start,
 				 int disjunction)
 {
-    char *s = get_name(rule->opts[start]);
+    char *s;
     int i;
 
+    G_asprintf(&s, "<%s>", get_name(rule->opts[start]));
+
     for (i = start + 1; i < rule->count - 1; i++) {
 	char *s0 = s;
 	char *ss = get_name(rule->opts[i]);
 	s = NULL;
-	G_asprintf(&s, "%s>, <%s", s0, ss);
+	G_asprintf(&s, "%s, <%s>", s0, ss);
 	G_free(s0);
 	G_free(ss);
     }
@@ -176,7 +178,7 @@ static const char *describe_rule(const struct rule *rule, int start,
 	char *s0 = s;
 	char *ss = get_name(rule->opts[i]);
 	s = NULL;
-	G_asprintf(&s, disjunction ? _("<%s> or <%s>") : _("<%s> and <%s>"), s0, ss);
+	G_asprintf(&s, disjunction ? _("%s or <%s>") : _("%s and <%s>"), s0, ss);
 	G_free(s0);
 	G_free(ss);
     }
@@ -270,7 +272,7 @@ static void check_requires(const struct rule *rule)
             G_asprintf(&err, _("Option <%s> requires at least one of %s"),
                        get_name(rule->opts[0]), describe_rule(rule, 1, 1));
         else
-            G_asprintf(&err, _("Option <%s> requires <%s>"),
+            G_asprintf(&err, _("Option <%s> requires %s"),
                        get_name(rule->opts[0]), describe_rule(rule, 1, 1));
         append_error(err);
     }