Message ID | 1446057425-16891-17-git-send-email-ehabkost@redhat.com |
---|---|
State | New |
Headers | show |
Eduardo Habkost <ehabkost@redhat.com> writes: > Make it a shorter and simpler phrase. > > Signed-off-by: Eduardo Habkost <ehabkost@redhat.com> > --- > vl.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/vl.c b/vl.c > index 6b831eb..67f75da 100644 > --- a/vl.c > +++ b/vl.c > @@ -2276,8 +2276,8 @@ static int parse_fw_cfg(void *opaque, QemuOpts *opts, Error **errp) > return -1; > } > if (strncmp(name, "opt/", 4) != 0) { > - error_report("warning: externally provided fw_cfg item names " > - "should be prefixed with \"opt/\"!"); > + error_report("warning: no \"opt/\" prefix on fw_cfg item name '%s'", > + name); > } > if (nonempty_str(str)) { > size = strlen(str); /* NUL terminator NOT included in fw_cfg blob */ The loss of the '!' is an improvement. The rest perhaps not so much.
diff --git a/vl.c b/vl.c index 6b831eb..67f75da 100644 --- a/vl.c +++ b/vl.c @@ -2276,8 +2276,8 @@ static int parse_fw_cfg(void *opaque, QemuOpts *opts, Error **errp) return -1; } if (strncmp(name, "opt/", 4) != 0) { - error_report("warning: externally provided fw_cfg item names " - "should be prefixed with \"opt/\"!"); + error_report("warning: no \"opt/\" prefix on fw_cfg item name '%s'", + name); } if (nonempty_str(str)) { size = strlen(str); /* NUL terminator NOT included in fw_cfg blob */
Make it a shorter and simpler phrase. Signed-off-by: Eduardo Habkost <ehabkost@redhat.com> --- vl.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)