On Tue, Oct 30, 2012 at 11:08:52PM +0200, Zeeshan Ali (Khattak) wrote:
> diff --git a/tools/osinfo-db-validate.c b/tools/osinfo-db-validate.c
> index f5855d1..2760b00 100644
> --- a/tools/osinfo-db-validate.c
> +++ b/tools/osinfo-db-validate.c
> @@ -33,7 +33,7 @@ static gboolean verbose = FALSE;
>
> static const GOptionEntry entries[] = {
> { "verbose", 'v', 0, G_OPTION_ARG_NONE, (void*)&verbose,
> - "Verbose progress information", NULL, },
> + N_("Verbose progress information"), NULL, },
> { NULL, 0, 0, 0, NULL, NULL, NULL }
> };
>
> @@ -255,12 +256,13 @@ gint main(gint argc, gchar **argv)
>
> g_type_init();
>
> - context = g_option_context_new("- Validate XML documents ");
> + context = g_option_context_new(_("- Validate XML documents "));
>
> - g_option_context_add_main_entries(context, entries, NULL);
> + g_option_context_add_main_entries(context, entries, GETTEXT_PACKAGE);
This change is enough to get everything translated, no need to add a call
to g_option_context_set_translation_domain ?
> diff --git a/tools/osinfo-query.c b/tools/osinfo-query.c
> index e68be90..eff323a 100644
> --- a/tools/osinfo-query.c
> +++ b/tools/osinfo-query.c
> @@ -287,10 +287,10 @@ static gboolean print_results_text(OsinfoList *list,
> g_print(" | ");
> first = FALSE;
>
> - if (strlen(labels[i].label) > labels[i].width)
> + if (strlen(gettext(labels[i].label)) > labels[i].width)
> pad = 0;
> else
> - pad = labels[i].width - strlen(labels[i].label);
> + pad = labels[i].width - strlen(gettext(labels[i].label));
I'd have a slight preference for _() here rather than gettext(), but I'm
fine either way.
Rest of the patch still looks good,
Christophe
Attachment:
pgpCcETHenjej.pgp
Description: PGP signature
