Eric Blake <ebl...@redhat.com> writes:

> On 12/17/2015 09:50 AM, Markus Armbruster wrote:
>> The arguments of error_report() should yield a short error string
>> without newlines.
>> 
>> A few places try to print additional help after the error message by
>> embedding newlines in the error string.  That's nice, but let's do it
>> the right way.  Commit 474c213 cleaned up some, but they keep coming
>> back.  Offenders tracked down with the Coccinelle semantic patch from
>> commit 312fd5f.
>
> Not sure if you'll have any stragglers from 1/23 here, after fixing 16/23.

Just one:
    qemu-nbd.c:604:28:"Invalid number of argument.\n" "Try `%s --help' for more 
information."

>> 
>> Cc: Laszlo Ersek <ler...@redhat.com>
>> Cc: Pavel Fedin <p.fe...@samsung.com>
>> Signed-off-by: Markus Armbruster <arm...@pond.sub.org>
>> Reviewed-by: Laszlo Ersek <ler...@redhat.com>
>> ---
>>  hw/i386/pc.c | 4 ++--
>>  kvm-all.c    | 6 +++---
>>  2 files changed, 5 insertions(+), 5 deletions(-)
>> 
>
> Reviewed-by: Eric Blake <ebl...@redhat.com>

Thanks!

Reply via email to