Message ID | 20170713163219.9024-13-marcandre.lureau@redhat.com |
---|---|
State | New |
Headers | show |
On 07/13/2017 01:32 PM, Marc-André Lureau wrote: > I used the clang-tidy qemu-round check to generate the fix: > https://github.com/elmarco/clang-tools-extra > > Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com> Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org> > --- > ui/cursor.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/ui/cursor.c b/ui/cursor.c > index 5155b392e8..2e2fe13fa6 100644 > --- a/ui/cursor.c > +++ b/ui/cursor.c > @@ -118,7 +118,7 @@ void cursor_put(QEMUCursor *c) > > int cursor_get_mono_bpl(QEMUCursor *c) > { > - return (c->width + 7) / 8; > + return DIV_ROUND_UP(c->width, 8); > } > > void cursor_set_mono(QEMUCursor *c, >
On 07/13/2017 06:32 AM, Marc-André Lureau wrote: > I used the clang-tidy qemu-round check to generate the fix: > https://github.com/elmarco/clang-tools-extra > > Signed-off-by: Marc-André Lureau<marcandre.lureau@redhat.com> > --- > ui/cursor.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Reviewed-by: Richard Henderson <rth@twiddle.net> r~
diff --git a/ui/cursor.c b/ui/cursor.c index 5155b392e8..2e2fe13fa6 100644 --- a/ui/cursor.c +++ b/ui/cursor.c @@ -118,7 +118,7 @@ void cursor_put(QEMUCursor *c) int cursor_get_mono_bpl(QEMUCursor *c) { - return (c->width + 7) / 8; + return DIV_ROUND_UP(c->width, 8); } void cursor_set_mono(QEMUCursor *c,
I used the clang-tidy qemu-round check to generate the fix: https://github.com/elmarco/clang-tools-extra Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com> --- ui/cursor.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)