On Thu, 2018-06-14 at 05:41 -0400, Frediano Ziglio wrote:
> >
> > 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) ?
I didn't know it was unnecessary. Perhaps another commit taking care of
that in all the syslog() calls, there are plenty "\n" there?
> Otherwise,
> Acked-by: Frediano Ziglio <fziglio@xxxxxxxxxx>
>
> Frediano
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel