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!