Re: [Qemu-devel] [PATCH v2 18/23] vmdk: Clean up "Invalid extent lines" error message

2015-12-18 Thread Markus Armbruster
Fam Zheng writes: > On Thu, 12/17 17:49, Markus Armbruster wrote: >> vmdk_parse_extents() reports parse errors like this: >> >> error_setg(errp, "Invalid extent lines:\n%s", p); >> >> where p points to the beginning of the malformed line in the image >> descriptor. This

[Qemu-devel] [PATCH v2 18/23] vmdk: Clean up "Invalid extent lines" error message

2015-12-17 Thread Markus Armbruster
vmdk_parse_extents() reports parse errors like this: error_setg(errp, "Invalid extent lines:\n%s", p); where p points to the beginning of the malformed line in the image descriptor. This results in a multi-line error message Invalid extent lines: Error messages should not

Re: [Qemu-devel] [PATCH v2 18/23] vmdk: Clean up "Invalid extent lines" error message

2015-12-17 Thread Eric Blake
On 12/17/2015 09:49 AM, Markus Armbruster wrote: > vmdk_parse_extents() reports parse errors like this: > > error_setg(errp, "Invalid extent lines:\n%s", p); > > where p points to the beginning of the malformed line in the image > descriptor. This results in a multi-line error message > >

Re: [Qemu-devel] [PATCH v2 18/23] vmdk: Clean up "Invalid extent lines" error message

2015-12-17 Thread Fam Zheng
On Thu, 12/17 17:49, Markus Armbruster wrote: > vmdk_parse_extents() reports parse errors like this: > > error_setg(errp, "Invalid extent lines:\n%s", p); > > where p points to the beginning of the malformed line in the image > descriptor. This results in a multi-line error message > >