On 11/17/2010 01:39 PM, Matthias Bolte wrote:
> Include locale.h for setlocale().
> 
> Revert the usage string back to it's original form.
> 
> Use puts() instead of fputs(), as fputs() expects a FILE*.
> 
> Add closing parenthesis to some vah_error() calls.
> 
> Use argv[0] instead of an undefined argv0.
> ---
>  src/security/virt-aa-helper.c |   39 ++++++++++++++++++++-------------------
>  1 files changed, 20 insertions(+), 19 deletions(-)

ACK.  And that's what I get for passing 'make syntax-check' on Fedora,
but not testing on ubuntu before actually pushing.  :(

Sorry, and I'll go crawl back in my hole now.  ;)

>  {
> -    fprintf(stdout, _("\n%s [options] [< def.xml]\n\n
> -  Options:\n
> -    -a | --add                     load profile\n

This would have worked if I'd used line continuations (in fact, that's
what coreutils does, to get a good feel for where the 80 column boundary is:

  fprintf(stdout, _("\
...\n\
...\n\
", ...);

but those missing \ got me.

> +    printf(_("\n%s [options] [< def.xml]\n\n"
> +            "  Options:\n"
> +            "    -a | --add                     load profile\n"

But this approach works as well.

> @@ -289,7 +290,7 @@ update_include_file(const char *include_file, const char 
> *included_files,
>      }
>  
>      if (VIR_CLOSE(fd) != 0) {
> -        vah_error(NULL, 0, _("failed to close or write to profile")_;
> +        vah_error(NULL, 0, _("failed to close or write to profile"));

I can't touch type very well, can I; my only excuse is that ) and _ are
neighbors.

> @@ -1140,7 +1141,7 @@ main(int argc, char **argv)
>      if (setlocale(LC_ALL, "") == NULL ||
>          bindtextdomain(PACKAGE, LOCALEDIR) == NULL ||
>          textdomain(PACKAGE) == NULL) {
> -        fprintf(stderr, _("%s: initialization failed\n"), argv0);
> +        fprintf(stderr, _("%s: initialization failed\n"), argv[0]);

copy-and-paste bytes again. :(

-- 
Eric Blake   ebl...@redhat.com    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Reply via email to