This patch shows the way, but is obviously incomplete as it works only
for "nobracket" version. Actually, I think the code should first build
the unbracketed output string and then do something like

if (!nobracket) {
        const char *to_free = v->s;
        v->s = xstrfmt("[%s]", v->s);
        free(to_free);
}

so we don't have to worry about brackets anywhere else in the code.

Signed-off-by: Matthieu Moy <matthieu....@imag.fr>
---
 ref-filter.c | 28 ++++++++++++++++++++++++----
 ref-filter.h |  3 +++
 2 files changed, 27 insertions(+), 4 deletions(-)

diff --git a/ref-filter.c b/ref-filter.c
index 7932c21..c2ee8c9 100644
--- a/ref-filter.c
+++ b/ref-filter.c
@@ -15,6 +15,26 @@
 #include "version.h"
 #include "wt-status.h"
 
+static struct ref_msg {
+       const char *gone;
+       const char *ahead;
+       const char *behind;
+       const char *ahead_behind;
+} msgs = {
+       "gone",
+       "ahead %d",
+       "behind %d",
+       "ahead %d, behind %d"
+};
+
+void setup_ref_filter_porcelain_msg(void)
+{
+       msgs.gone = _("gone");
+       msgs.ahead = _("ahead %d");
+       msgs.behind = _("behind %d");
+       msgs.ahead_behind = _("ahead %d, behind %d");
+}
+
 typedef enum { FIELD_STR, FIELD_ULONG, FIELD_TIME } cmp_type;
 
 static struct {
@@ -1124,7 +1144,7 @@ static void populate_value(struct ref_array_item *ref)
                                if (stat_tracking_info(branch, &num_ours,
                                                       &num_theirs, NULL)) {
                                        if (nobracket)
-                                               v->s = "gone";
+                                               v->s = msgs.gone;
                                        else
                                                v->s = "[gone]";
                                        continue;
@@ -1134,17 +1154,17 @@ static void populate_value(struct ref_array_item *ref)
                                        v->s = "";
                                else if (!num_ours) {
                                        if (nobracket)
-                                               v->s = xstrfmt("behind %d", 
num_theirs);
+                                               v->s = xstrfmt(msgs.behind, 
num_theirs);
                                        else
                                                v->s = xstrfmt("[behind %d]", 
num_theirs);
                                } else if (!num_theirs) {
                                        if (nobracket)
-                                               v->s = xstrfmt("ahead %d", 
num_ours);
+                                               v->s = xstrfmt(msgs.ahead, 
num_ours);
                                        else
                                                v->s = xstrfmt("[ahead %d]", 
num_ours);
                                } else {
                                        if (nobracket)
-                                               v->s = xstrfmt("ahead %d, 
behind %d",
+                                               v->s = 
xstrfmt(msgs.ahead_behind,
                                                               num_ours, 
num_theirs);
                                        else
                                                v->s = xstrfmt("[ahead %d, 
behind %d]",
diff --git a/ref-filter.h b/ref-filter.h
index 0014b92..2cce02c 100644
--- a/ref-filter.h
+++ b/ref-filter.h
@@ -112,4 +112,7 @@ int parse_opt_merge_filter(const struct option *opt, const 
char *arg, int unset)
 /*  Get the current HEAD's description */
 char *get_head_description(void);
 
+/*  Set up translated strings in the output. */
+void setup_ref_filter_porcelain_msg(void);
+
 #endif /*  REF_FILTER_H  */
-- 
2.6.0.rc2.24.gb06d8e9.dirty

--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to