[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] fw_cfg: print error message when reading splash
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH] fw_cfg: print error message when reading splashfile failed |
Date: |
Wed, 31 Oct 2018 16:36:00 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.1 (gnu/linux) |
Philippe Mathieu-Daudé <address@hidden> writes:
> On 24/10/18 7:12, Li Qiang wrote:
>> Also remove unnecessary 'res' variable.
>>
>> Signed-off-by: Li Qiang <address@hidden>
>> ---
>> hw/nvram/fw_cfg.c | 7 +++----
>> 1 file changed, 3 insertions(+), 4 deletions(-)
>>
>> diff --git a/hw/nvram/fw_cfg.c b/hw/nvram/fw_cfg.c
>> index 946f765..f4a52d8 100644
>> --- a/hw/nvram/fw_cfg.c
>> +++ b/hw/nvram/fw_cfg.c
>> @@ -68,15 +68,14 @@ static char *read_splashfile(char *filename, gsize
>> *file_sizep,
>> int *file_typep)
>> {
>> GError *err = NULL;
>> - gboolean res;
>> gchar *content;
>> int file_type;
>> unsigned int filehead;
>> int bmp_bpp;
>> - res = g_file_get_contents(filename, &content, file_sizep,
>> &err);
>> - if (res == FALSE) {
>> - error_report("failed to read splash file '%s'", filename);
>> + if (!g_file_get_contents(filename, &content, file_sizep, &err)) {
>> + error_report("failed to read splash file '%s', %s",
>
> Can you use a column like the rest of the codebase?
>
> i.e.: "failed to read splash file '%s': %s"
Yes, please.
Also, my remark on your "[PATCH] vl.c: print error message if loading
fw_cfg file failed" applies to this patch as well. Please improve the
commit message.
> The maintainer taking this patch can do this minor change, so:
> Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
>
>> + filename, err->message);
>> g_error_free(err);
>> return NULL;
>> }
>>