+-- On Wed, 20 Jan 2016, Alex Williamson wrote --+
| > > + error_printf("Warning : Device at %s is known to cause
| > > system instability issues during option rom execution. Proceeding
| > > anyway since user specified romfile\n",
| > > + vdev->vbasedev.name);
| >
| > This error message should really be split across lines like
| > the original code was. Likewise for other cases in this patch
|
| the source to find an error message, it should work. Breaking printed
| error messages at arbitrary lengths to fit an 80 column window makes
| that more difficult. Is there sufficient passion for a hard 80 column
| limit on printed strings in QEMU to ignore that useful property?
I'd say +1 for breaking it at 80'th column. IMO it helps with readability in
editor or tools like less(1). As for grep, it can still work with a
substring/pattern from the error message.
--
Prasad J Pandit / Red Hat Product Security Team
47AF CE69 3A90 54AA 9045 1053 DD13 3D32 FE5B 041F
