Hi Alex, On 11/19/20 4:01 AM, Alex Chen wrote: > We should use printf format specifier "%u" instead of "%d" for > argument of type "unsigned int". > > Reported-by: Euler Robot <euler.ro...@huawei.com> > Signed-off-by: Alex Chen <alex.c...@huawei.com> > --- > hw/display/vmware_vga.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/hw/display/vmware_vga.c b/hw/display/vmware_vga.c > index bef0d7d69a..f93bbe15c2 100644 > --- a/hw/display/vmware_vga.c > +++ b/hw/display/vmware_vga.c > @@ -534,7 +534,7 @@ static inline void vmsvga_cursor_define(struct > vmsvga_state_s *s, > #endif > break; > default: > - fprintf(stderr, "%s: unhandled bpp %d, using fallback cursor\n", > + fprintf(stderr, "%s: unhandled bpp %u, using fallback cursor\n", > __func__, c->bpp);
The format fix is correct, but since you change that line, "fprintf(stderr)" is old code, nowadays we prefer get rid of it by using warn_report() in place (see "qemu/error-report.h"). Preferably using warn_report(): Reviewed-by: Philippe Mathieu-Daudé <phi...@redhat.com> > cursor_put(qc); > qc = cursor_builtin_left_ptr(); >