Patchwork qxl: Don't convert from size_t to int and back in qxl_cursor()

login
register
mail settings
Submitter Markus Armbruster
Date Nov. 9, 2011, 8:52 a.m.
Message ID <1320828775-16828-1-git-send-email-armbru@redhat.com>
Download mbox | patch
Permalink /patch/124491/
State New
Headers show

Comments

Markus Armbruster - Nov. 9, 2011, 8:52 a.m.
Just for cleanliness; it would take a truly gigantic cursor to break.

Signed-off-by: Markus Armbruster <armbru@redhat.com>
---
 hw/qxl-render.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
Stefan Hajnoczi - Nov. 9, 2011, 10:13 a.m.
On Wed, Nov 09, 2011 at 09:52:55AM +0100, Markus Armbruster wrote:
> Just for cleanliness; it would take a truly gigantic cursor to break.
> 
> Signed-off-by: Markus Armbruster <armbru@redhat.com>
> ---
>  hw/qxl-render.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)

Thanks, applied to the trivial patches tree:
http://repo.or.cz/w/qemu/stefanha.git/shortlog/refs/heads/trivial-patches

Stefan

Patch

diff --git a/hw/qxl-render.c b/hw/qxl-render.c
index 2c51ba9..133d093 100644
--- a/hw/qxl-render.c
+++ b/hw/qxl-render.c
@@ -157,7 +157,7 @@  static QEMUCursor *qxl_cursor(PCIQXLDevice *qxl, QXLCursor *cursor)
 {
     QEMUCursor *c;
     uint8_t *image, *mask;
-    int size;
+    size_t size;
 
     c = cursor_alloc(cursor->header.width, cursor->header.height);
     c->hot_x = cursor->header.hot_spot_x;