Changeset 3535Tweet

Timestamp:

May 28, 2009, 3:54:50 AM (11 years ago)

Author:

Sam Hocevar

Message:

Beginning of the optim...

File:

• libpipi/trunk/examples/img2twit.cpp (modified) (5 diffs)

libpipi/trunk/examples/img2twit.cpp

@@ -9 +9 @@
  *  To Public License, Version 2, as published by Sam Hocevar. See
  *  http://sam.zoy.org/wtfpl/COPYING for more details.
+ */
+
+/* TODO:
+ * - remove the complicated stuff from get_point/set_point, it's only
+ *   the final packing that really matters.
  */
 
@@ -111 +116 @@
 
 #define RANGE_XY (RANGE_Y*RANGE_X)
+#define RANGE_XYXY (RANGE_XY*RANGE_XY)
+//#define RANGE_XYXY (RANGE_Y*RANGE_X*(RANGE_Y*RANGE_X+1)/2)
 #define RANGE_RXY (RANGE_Y*RANGE_X*RANGE_R)
 #define RANGE_GRXY (RANGE_Y*RANGE_X*RANGE_R*RANGE_G)
@@ -549 +556 @@
     set_point(npoints, x, y, r, g, b, s);
     npoints++;
+}
+
+static uint32_t pack_coords(int x1, int y1, int x2, int y2)
+{
+    return ((y2 * RANGE_X + x2) * RANGE_Y + y1) * RANGE_X + x1;
+}
+
+static void unpack_coords(uint32_t pack, int *x1, int *y1, int *x2, int *y2)
+{
+    *x1 = pack % RANGE_X; pack /= RANGE_X;
+    *y1 = pack % RANGE_Y; pack /= RANGE_Y;
+    *x2 = pack % RANGE_X; pack /= RANGE_X;
+    *y2 = pack % RANGE_Y;
 }
 
@@ -1274 +1294 @@
         {
 #if POINTS_PER_CELL == 2
+            int x1, y1, x2, y2;
+            x1 = (points[i] / RANGE_Y) % RANGE_X;
+            y1 = points[i] % RANGE_Y;
+            x2 = (points[i + 1] / RANGE_Y) % RANGE_X;
+            y2 = points[i + 1] % RANGE_Y;
+            uint32_t pack = pack_coords(x1, y1, x2, y2);
+
             b.push(points[i] / RANGE_XY, RANGE_SBGR);
-            b.push(points[i] % RANGE_XY, RANGE_XY);
             b.push(points[i + 1] / RANGE_XY, RANGE_SBGR);
-            b.push(points[i + 1] % RANGE_XY, RANGE_XY);
+            b.push(pack, RANGE_XYXY);
 #else
             b.push(points[i], RANGE_SBGRXY);
@@ -1296 +1322 @@
         {
 #if POINTS_PER_CELL == 2
-            uint32_t c1 = b.pop(RANGE_XY);
+            uint32_t pack = b.pop(RANGE_XYXY);
+            uint32_t p2 = b.pop(RANGE_SBGR);
             uint32_t p1 = b.pop(RANGE_SBGR);
-            uint32_t c2 = b.pop(RANGE_XY);
-            uint32_t p2 = b.pop(RANGE_SBGR);
-            points[i * 2 + 1] = p1 * RANGE_XY + c1;
-            points[i * 2] = p2 * RANGE_XY + c2;
+            int x1, y1, x2, y2;
+            unpack_coords(pack, &x1, &y1, &x2, &y2);
+            points[i * 2] = p1 * RANGE_XY + x1 * RANGE_Y + y1;
+            points[i * 2 + 1] = p2 * RANGE_XY + x2 * RANGE_Y + y2;
 #else
             points[i] = b.pop(RANGE_SBGRXY);