[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-trivial] [PATCH for-1.4 v2 1/6] error: Clean up error strings
|
From: |
Luiz Capitulino |
|
Subject: |
Re: [Qemu-trivial] [PATCH for-1.4 v2 1/6] error: Clean up error strings with embedded newlines |
|
Date: |
Wed, 13 Feb 2013 17:28:40 -0200 |
On Fri, 08 Feb 2013 20:48:18 +0100
Markus Armbruster <address@hidden> wrote:
> It's a cleanup. It's only user-visible effect is getting rid of an
> extra newline on stderr. I'm fixing those globally. Tiny improvement
> in user experience, but next to no risk, thus proposed for 1.4. Since I
> need to touch this call anyway for that, I can just as well clean up the
> API abuse. The alternative is to leave the abuse alone and just strip
> the final newline. That would make me sad.
>
> If you hate the API, fix it. Don't make me not fix abuses of it :)
I honestly would prefer to defer this and fix the API once and for all.
But it's not worth to discuss this anymore, as this patch has already
been merged.
[Qemu-trivial] [PATCH for-1.4 v2 4/6] qemu-option: Disable two helpful messages that got broken recently, Markus Armbruster, 2013/02/08
[Qemu-trivial] [PATCH for-1.4 v2 5/6] vl: Drop redundant "parse error" reports, Markus Armbruster, 2013/02/08
[Qemu-trivial] [PATCH for-1.4 v2 6/6] vl: Exit unsuccessfully on option argument syntax error, Markus Armbruster, 2013/02/08