Changeset 237Tweet

Timestamp:

Nov 30, 2003, 3:11:37 AM (17 years ago)

Author:

Sam Hocevar

Message:

• src/bitmap.c examples/view.c: + Fixed an endianness issue in caca_draw_bitmap(). + Code cleanup.

Location:

libcaca/trunk

Files:

• examples/view.c (modified) (2 diffs)
• src/bitmap.c (modified) (14 diffs)

libcaca/trunk/examples/view.c

@@ -216 +216 @@
             if(yn + y > h) y = h - yn;
             newbitmap = caca_create_bitmap(32, 2 * xn, 2 * yn, 4 * w,
-                                       0x00ff0000, 0x0000ff00, 0x000000ff);
+                                       0x0000ff00, 0x00ff0000, 0xff000000);
             caca_draw_bitmap(0, 0, ww - 1, wh - 1, newbitmap,
                              pixels + 4 * (x - xn) + 4 * w * (y - yn));
@@ -299 +299 @@
     /* Create the libcaca bitmap */
     bitmap = caca_create_bitmap(32, w, h, 4 * w,
-                                0x00ff0000, 0x0000ff00, 0x000000ff);
+                                0x0000ff00, 0x00ff0000, 0xff000000);
     if(!bitmap)
     {

libcaca/trunk/src/bitmap.c

@@ -43 +43 @@
 #include "caca_internals.h"
 
-static void mask2shift(int, int *, int *);
-
-static void get_rgb_default(struct caca_bitmap *, unsigned char *,
-                            int, int, int *, int *, int *);
+static void mask2shift(unsigned int, int *, int *);
+
+static void get_rgb_default(struct caca_bitmap *, uint8_t *, int, int,
+                            unsigned int *, unsigned int *, unsigned int *);
 static void rgb2hsv_default(int, int, int, int *, int *, int *);
 
 /* Dithering methods */
 static void init_no_dither(int);
-static int get_no_dither(void);
+static unsigned int get_no_dither(void);
 static void increment_no_dither(void);
 
 static void init_ordered4_dither(int);
-static int get_ordered4_dither(void);
+static unsigned int get_ordered4_dither(void);
 static void increment_ordered4_dither(void);
 
 static void init_ordered8_dither(int);
-static int get_ordered8_dither(void);
+static unsigned int get_ordered8_dither(void);
 static void increment_ordered8_dither(void);
 
 static void init_random_dither(int);
-static int get_random_dither(void);
+static unsigned int get_random_dither(void);
 static void increment_random_dither(void);
 
@@ -70 +70 @@
 
 static void (*_init_dither) (int) = init_no_dither;
-static int (*_get_dither) (void) = get_no_dither;
+static unsigned int (*_get_dither) (void) = get_no_dither;
 static void (*_increment_dither) (void) = increment_no_dither;
 
@@ -119 +119 @@
 };
 
-static void mask2shift(int mask, int *right, int *left)
+static void mask2shift(unsigned int mask, int *right, int *left)
 {
     int rshift = 0, lshift = 0;
-    *right = *left = 0;
 
     if(!mask)
+    {
+        *right = *left = 0;
         return;
+    }
 
     while(!(mask & 1))
@@ -142 +144 @@
 }
 
+#include <stdio.h>
 struct caca_bitmap *caca_create_bitmap(int bpp, int w, int h, int pitch,
                                        int rmask, int gmask, int bmask)
@@ -173 +176 @@
         mask2shift(bmask, &bitmap->bright, &bitmap->bleft);
     }
+fprintf(stderr, "shifts: %i %i %i %i %i %i\n", bitmap->rright, bitmap->rleft, bitmap->gright, bitmap->gleft, bitmap->bright, bitmap->bleft);
 
     /* In 8 bpp mode, default to a grayscale palette */
@@ -219 +223 @@
 }
 
-static void get_rgb_default(struct caca_bitmap *bitmap, unsigned char *pixels,
-                            int x, int y, int *r, int *g, int *b)
-{
-    int bits;
+static void get_rgb_default(struct caca_bitmap *bitmap, uint8_t *pixels,
+                            int x, int y, unsigned int *r,
+                            unsigned int *g, unsigned int *b)
+{
+    unsigned int bits;
 
     pixels += (bitmap->bpp / 8) * x + bitmap->pitch * y;
@@ -229 +234 @@
     {
         case 4:
-            bits = *(uint32_t *)(pixels + 0);
+            bits = ((uint32_t)pixels[0] << 24) |
+                   ((uint32_t)pixels[1] << 16) |
+                   ((uint32_t)pixels[2] << 8) |
+                   ((uint32_t)pixels[3]);
             break;
         case 3:
-            bits = (pixels[0] << 16) | (pixels[1] << 8) | (pixels[2]);
+            bits = ((uint32_t)pixels[0] << 16) |
+                   ((uint32_t)pixels[1] << 8) |
+                   ((uint32_t)pixels[2]);
             break;
         case 2:
-            bits = *(uint16_t *)(pixels + 0);
+            bits = ((uint16_t)pixels[0] << 8) |
+                   ((uint16_t)pixels[1]);
             break;
         case 1:
@@ -362 +373 @@
         {
             int ch;
-            int hue, sat, val, r, g, b, R, G, B;
+            unsigned int hue, sat, val, r, g, b, R, G, B;
             int fromx = w * (x - x1) / (x2 - x1 + 1);
             int fromy = h * (y - y1) / (y2 - y1 + 1);
@@ -416 +427 @@
 }
 
-static int get_no_dither(void)
+static unsigned int get_no_dither(void)
 {
     return 0x80;
@@ -433 +444 @@
                           -2, -7, -8,  3,
                            4, -3, -4, -7};*/
-static int *ordered4_table;
-static int ordered4_index;
+static unsigned int *ordered4_table;
+static unsigned int ordered4_index;
 
 static void init_ordered4_dither(int line)
 {
-    static int dither4x4[] =
+    static unsigned int dither4x4[] =
     {
         0x00, 0x80, 0x20, 0xa0,
@@ -450 +461 @@
 }
 
-static int get_ordered4_dither(void)
+static unsigned int get_ordered4_dither(void)
 {
     return ordered4_table[ordered4_index];
@@ -463 +474 @@
  * Ordered 8 dithering
  */
-static int *ordered8_table;
-static int ordered8_index;
+static unsigned int *ordered8_table;
+static unsigned int ordered8_index;
 
 static void init_ordered8_dither(int line)
 {
-    static int dither8x8[] =
+    static unsigned int dither8x8[] =
     {
         0x00, 0x80, 0x20, 0xa0, 0x08, 0x88, 0x28, 0xa8,
@@ -484 +495 @@
 }
 
-static int get_ordered8_dither(void)
+static unsigned int get_ordered8_dither(void)
 {
     return ordered8_table[ordered8_index];
@@ -502 +513 @@
 }
 
-static int get_random_dither(void)
+static unsigned int get_random_dither(void)
 {
     return caca_rand(0x00, 0xff);