[PATCH spice-common] canvas-base: Do not attempt useless cast on stride adjustment

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



memmove already deal with any alignment so there's no
reason to have row byte pointer cast to uint32_t.

Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx>
---
 common/canvas_base.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/common/canvas_base.c b/common/canvas_base.c
index 5815e9c..5b3649d 100644
--- a/common/canvas_base.c
+++ b/common/canvas_base.c
@@ -522,9 +522,9 @@ static void canvas_fix_alignment(uint8_t *bits,
         int row;
         uint8_t *dest = bits;
         for (row = height - 1; row > 0; --row) {
-            uint32_t *dest_aligned, *dest_misaligned;
-            dest_aligned = SPICE_ALIGNED_CAST(uint32_t *,dest + stride_pixman*row);
-            dest_misaligned = SPICE_UNALIGNED_CAST(uint32_t*,dest + stride_encoded*row);
+            uint8_t *dest_aligned, *dest_misaligned;
+            dest_aligned = dest + stride_pixman*row;
+            dest_misaligned = dest + stride_encoded*row;
             memmove(dest_aligned, dest_misaligned, stride_encoded);
         }
     }
-- 
2.9.4

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]