Re: [PATCH spice-streaming-agent 4/9] Improve the invalid option argument (-c) error message

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

 



> 
> Signed-off-by: Lukáš Hrázký <lhrazky@xxxxxxxxxx>
> ---
>  src/spice-streaming-agent.cpp | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/spice-streaming-agent.cpp b/src/spice-streaming-agent.cpp
> index ca10d3b..d5ed034 100644
> --- a/src/spice-streaming-agent.cpp
> +++ b/src/spice-streaming-agent.cpp
> @@ -429,7 +429,7 @@ int main(int argc, char* argv[])
>          case 'c': {
>              char *p = strchr(optarg, '=');
>              if (p == NULL) {
> -                syslog(LOG_ERR, "wrong 'c' argument %s\n", optarg);
> +                syslog(LOG_ERR, "Invalid '-c' argument value: %s\n",
> optarg);
>                  usage(argv[0]);
>              }
>              *p++ = '\0';

Why not removing also the not necessary line terminator (\n) ?

Otherwise,
Acked-by: Frediano Ziglio <fziglio@xxxxxxxxxx>

Frediano
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux Virtualization]     [Linux Virtualization]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]