iliaa           Tue Mar 18 13:29:10 2003 EDT

  Modified files:              
    /php4/ext/gd/libgd  gdxpm.c 
  Log:
  Cleanup.
  
  
Index: php4/ext/gd/libgd/gdxpm.c
diff -u php4/ext/gd/libgd/gdxpm.c:1.4 php4/ext/gd/libgd/gdxpm.c:1.5
--- php4/ext/gd/libgd/gdxpm.c:1.4       Sun Dec  1 06:43:54 2002
+++ php4/ext/gd/libgd/gdxpm.c   Tue Mar 18 13:29:10 2003
@@ -15,123 +15,125 @@
 
 #include <X11/xpm.h>
 
-gdImagePtr
-gdImageCreateFromXpm (char *filename)
+gdImagePtr gdImageCreateFromXpm (char *filename)
 {
-  XpmInfo info;
-  XpmImage image;
-  int i, j, k, number;
-  char buf[5];
-  gdImagePtr im = 0;
-  char *apixel;
-  int *pointer;
-  int red = 0, green = 0, blue = 0;
-  int *colors;
-  int ret;
-
-  ret = XpmReadFileToXpmImage (filename, &image, &info);
-  if (ret != XpmSuccess)
-    return 0;
-
-  if (!(im = gdImageCreate (image.width, image.height)))
-    return 0;
-
-  number = image.ncolors;
-  colors = (int *) gdMalloc (sizeof (int) * number);
-  for (i = 0; i < number; i++)
-    {
-      switch (strlen (image.colorTable[i].c_color))
-       {
-       case 4:
-         buf[1] = '\0';
-         buf[0] = image.colorTable[i].c_color[1];
-         red = strtol (buf, NULL, 16);
-
-         buf[0] = image.colorTable[i].c_color[3];
-         green = strtol (buf, NULL, 16);
-
-         buf[0] = image.colorTable[i].c_color[5];
-         blue = strtol (buf, NULL, 16);
-         break;
-       case 7:
-         buf[2] = '\0';
-         buf[0] = image.colorTable[i].c_color[1];
-         buf[1] = image.colorTable[i].c_color[2];
-         red = strtol (buf, NULL, 16);
-
-         buf[0] = image.colorTable[i].c_color[3];
-         buf[1] = image.colorTable[i].c_color[4];
-         green = strtol (buf, NULL, 16);
-
-         buf[0] = image.colorTable[i].c_color[5];
-         buf[1] = image.colorTable[i].c_color[6];
-         blue = strtol (buf, NULL, 16);
-         break;
-       case 10:
-         buf[3] = '\0';
-         buf[0] = image.colorTable[i].c_color[1];
-         buf[1] = image.colorTable[i].c_color[2];
-         buf[2] = image.colorTable[i].c_color[3];
-         red = strtol (buf, NULL, 16);
-         red /= 64;
-
-         buf[0] = image.colorTable[i].c_color[4];
-         buf[1] = image.colorTable[i].c_color[5];
-         buf[2] = image.colorTable[i].c_color[6];
-         green = strtol (buf, NULL, 16);
-         green /= 64;
-
-         buf[0] = image.colorTable[i].c_color[7];
-         buf[1] = image.colorTable[i].c_color[8];
-         buf[2] = image.colorTable[i].c_color[9];
-         blue = strtol (buf, NULL, 16);
-         blue /= 64;
-         break;
-       case 13:
-         buf[4] = '\0';
-         buf[0] = image.colorTable[i].c_color[1];
-         buf[1] = image.colorTable[i].c_color[2];
-         buf[2] = image.colorTable[i].c_color[3];
-         buf[3] = image.colorTable[i].c_color[4];
-         red = strtol (buf, NULL, 16);
-         red /= 256;
-
-         buf[0] = image.colorTable[i].c_color[5];
-         buf[1] = image.colorTable[i].c_color[6];
-         buf[2] = image.colorTable[i].c_color[7];
-         buf[3] = image.colorTable[i].c_color[8];
-         green = strtol (buf, NULL, 16);
-         green /= 256;
-
-         buf[0] = image.colorTable[i].c_color[9];
-         buf[1] = image.colorTable[i].c_color[10];
-         buf[2] = image.colorTable[i].c_color[11];
-         buf[3] = image.colorTable[i].c_color[12];
-         blue = strtol (buf, NULL, 16);
-         blue /= 256;
-         break;
+       XpmInfo info;
+       XpmImage image;
+       int i, j, k, number;
+       char buf[5];
+       gdImagePtr im = 0;
+       char *apixel;
+       int *pointer;
+       int red = 0, green = 0, blue = 0;
+       int *colors;
+       int ret;
+
+       ret = XpmReadFileToXpmImage(filename, &image, &info);
+       if (ret != XpmSuccess) {
+               return 0;
        }
 
+       if (!(im = gdImageCreate(image.width, image.height))) {
+               return 0;
+       }
 
-      colors[i] = gdImageColorResolve (im, red, green, blue);
-      if (colors[i] == -1)
-       php_gd_error("ARRRGH\n");
-    }
-
-  apixel = (char *) gdMalloc (image.cpp + 1);
-  apixel[image.cpp] = '\0';
-
-  pointer = (int *) image.data;
-  for (i = 0; i < image.height; i++)
-    {
-      for (j = 0; j < image.width; j++)
-       {
-         k = *pointer++;
-         gdImageSetPixel (im, j, i, colors[k]);
+       number = image.ncolors;
+       colors = (int *) gdMalloc (sizeof (int) * number);
+       for (i = 0; i < number; i++) {
+               switch (strlen (image.colorTable[i].c_color)) {
+                       case 4:
+                               buf[1] = '\0';
+                               buf[0] = image.colorTable[i].c_color[1];
+                               red = strtol(buf, NULL, 16);
+
+                               buf[0] = image.colorTable[i].c_color[3];
+                               green = strtol(buf, NULL, 16);
+
+                               buf[0] = image.colorTable[i].c_color[5];
+                               blue = strtol(buf, NULL, 16);
+                               break;
+
+                       case 7:
+                               buf[2] = '\0';
+                               buf[0] = image.colorTable[i].c_color[1];
+                               buf[1] = image.colorTable[i].c_color[2];
+                               red = strtol(buf, NULL, 16);
+
+                               buf[0] = image.colorTable[i].c_color[3];
+                               buf[1] = image.colorTable[i].c_color[4];
+                               green = strtol(buf, NULL, 16);
+
+                               buf[0] = image.colorTable[i].c_color[5];
+                               buf[1] = image.colorTable[i].c_color[6];
+                               blue = strtol(buf, NULL, 16);
+                               break;
+
+                       case 10:
+                               buf[3] = '\0';
+                               buf[0] = image.colorTable[i].c_color[1];
+                               buf[1] = image.colorTable[i].c_color[2];
+                               buf[2] = image.colorTable[i].c_color[3];
+                               red = strtol(buf, NULL, 16);
+                               red /= 64;
+
+                               buf[0] = image.colorTable[i].c_color[4];
+                               buf[1] = image.colorTable[i].c_color[5];
+                               buf[2] = image.colorTable[i].c_color[6];
+                               green = strtol(buf, NULL, 16);
+                               green /= 64;
+
+                               buf[0] = image.colorTable[i].c_color[7];
+                               buf[1] = image.colorTable[i].c_color[8];
+                               buf[2] = image.colorTable[i].c_color[9];
+                               blue = strtol(buf, NULL, 16);
+                               blue /= 64;
+                               break;
+
+                       case 13:
+                               buf[4] = '\0';
+                               buf[0] = image.colorTable[i].c_color[1];
+                               buf[1] = image.colorTable[i].c_color[2];
+                               buf[2] = image.colorTable[i].c_color[3];
+                               buf[3] = image.colorTable[i].c_color[4];
+                               red = strtol(buf, NULL, 16);
+                               red /= 256;
+
+                               buf[0] = image.colorTable[i].c_color[5];
+                               buf[1] = image.colorTable[i].c_color[6];
+                               buf[2] = image.colorTable[i].c_color[7];
+                               buf[3] = image.colorTable[i].c_color[8];
+                               green = strtol(buf, NULL, 16);
+                               green /= 256;
+
+                               buf[0] = image.colorTable[i].c_color[9];
+                               buf[1] = image.colorTable[i].c_color[10];
+                               buf[2] = image.colorTable[i].c_color[11];
+                               buf[3] = image.colorTable[i].c_color[12];
+                               blue = strtol(buf, NULL, 16);
+                               blue /= 256;
+                               break;
+               }
+
+
+               colors[i] = gdImageColorResolve(im, red, green, blue);
+               if (colors[i] == -1) {
+                       php_gd_error("ARRRGH\n");
+               }
        }
-    }
-  gdFree (apixel);
-  gdFree (colors);
-  return (im);
+
+       apixel = (char *) gdMalloc(image.cpp + 1);
+       apixel[image.cpp] = '\0';
+
+       pointer = (int *) image.data;
+       for (i = 0; i < image.height; i++) {
+               for (j = 0; j < image.width; j++) {
+                       k = *pointer++;
+                       gdImageSetPixel(im, j, i, colors[k]);
+               }
+       }
+
+       gdFree(apixel);
+       gdFree(colors);
+       return im;
 }
 #endif



-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to