Re: [PATCH] util: Re-format literal strings in virXMLEmitWarning

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Oct 29, 2012 at 03:22:05PM +0100, Peter Krempa wrote:
> And drop a stray space at the end of the first line of the warning.
> ---
> Yes, some lines exceed 80 columns.
> ---
>  src/util/xml.c | 16 +++++++++-------
>  1 file changed, 9 insertions(+), 7 deletions(-)
> 
> diff --git a/src/util/xml.c b/src/util/xml.c
> index dad9227..30323b3 100644
> --- a/src/util/xml.c
> +++ b/src/util/xml.c
> @@ -796,13 +796,15 @@ static int virXMLEmitWarning(int fd,
>                               const char *cmd)
>  {
>      size_t len;
> -    const char *prologue = "<!--\n\
> -WARNING: THIS IS AN AUTO-GENERATED FILE. CHANGES TO IT ARE LIKELY TO BE \n\
> -OVERWRITTEN AND LOST. Changes to this xml configuration should be made using:\n\
> -  virsh ";
> -    const char *epilogue = "\n\
> -or other application using the libvirt API.\n\
> --->\n\n";
> +    const char *prologue =
> +        "<!--\n"
> +        "WARNING: THIS IS AN AUTO-GENERATED FILE. CHANGES TO IT ARE LIKELY TO BE\n"
> +        "OVERWRITTEN AND LOST. Changes to this xml configuration should be made using:\n"
> +        "  virsh ";
> +    const char *epilogue =
> +        "\n"
> +        "or other application using the libvirt API.\n"
> +        "-->\n\n";

ACK

Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|

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


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]