Store full-colour image with every screenshot

diff --git a/pctb/common.h b/pctb/common.h
index e4100f9d461bcfe216c061970016b4daa759936b..23b8ca33972d29bf589cae1799896c70408392bf 100644 (file)
--- a/pctb/common.h
+++ b/pctb/common.h
@@ -48,6 +48,7 @@
 
 typedef struct {
   int w,h;
 
 typedef struct {
   int w,h;

+  struct RgbImage *rgb;

   char d[];
 } CanonImage;
 
   char d[];
 } CanonImage;
 

diff --git a/pctb/convert.c b/pctb/convert.c
index c7c027adb05baa0c3ba3f879ef2a38c20a7298ec..8101fb920a40adafad02ab92ed4efd5181c2ae44 100644 (file)
--- a/pctb/convert.c
+++ b/pctb/convert.c
@@ -305,7 +305,7 @@ int main(int argc, char **argv) {
   }
   if (o_mode & mf_analyse) {
     if (o_flags & ff_needisland) {
   }
   if (o_mode & mf_analyse) {
     if (o_flags & ff_needisland) {

-      find_islandname(page0_rgbimage);
+      find_islandname(page_images[0]->rgb);

       if (o_flags & ff_printisland)
        printf("%s, %s\n", archipelago, island);
       sysassert(! setenv("YPPSC_ISLAND",island,1) );
       if (o_flags & ff_printisland)
        printf("%s, %s\n", archipelago, island);
       sysassert(! setenv("YPPSC_ISLAND",island,1) );

diff --git a/pctb/convert.h b/pctb/convert.h
index 80979ec2e885829f8d15faccd887c9c63a6812b0..83be1e8bced1ab8fce05d7233588595792f74530 100644 (file)
--- a/pctb/convert.h
+++ b/pctb/convert.h
@@ -118,7 +118,6 @@ void take_one_screenshot(void);
 #define MAX_PAGES 100
 extern CanonImage *page_images[MAX_PAGES];
 extern int npages;
 #define MAX_PAGES 100
 extern CanonImage *page_images[MAX_PAGES];
 extern int npages;

-RgbImage *page0_rgbimage;

 
 extern const char *ocean, *pirate;
 extern char *archipelago, *island;
 
 extern const char *ocean, *pirate;
 extern char *archipelago, *island;

diff --git a/pctb/pages.c b/pctb/pages.c
index 482c959c24261cf541ca0ea4e99b63ddf7e51e4e..b3ddceba81f921c2dc1d46661053e77da3215680 100644 (file)
--- a/pctb/pages.c
+++ b/pctb/pages.c
@@ -43,7 +43,6 @@
 
 CanonImage *page_images[MAX_PAGES];
 int npages;
 
 CanonImage *page_images[MAX_PAGES];
 int npages;

-RgbImage *page0_rgbimage;

 
 const char *ocean, *pirate;
 
 
 const char *ocean, *pirate;
 


@@ -349,7 +348,7 @@ static void wait_for_stability(Snapshot **output,
       last=*output; *output=0;
     } else if (!identical(*output,last)) {
       debugf("PAGING  wait_for_stability changed...\n");
       last=*output; *output=0;
     } else if (!identical(*output,last)) {
       debugf("PAGING  wait_for_stability changed...\n");

-      free(last); last=*output; *output=0;
+      free_snapshot(&last); last=*output; *output=0;

       nidentical=0;
       if (!with_keypress) {
        min_interval *= 3.0;
       nidentical=0;
       if (!with_keypress) {
        min_interval *= 3.0;


@@ -494,11 +493,10 @@ static void set_focus_commodity(void) {
   debugf("PAGING raise_and_set_focus done.\n");
 }
 
   debugf("PAGING raise_and_set_focus done.\n");
 }
 

-static CanonImage *convert_page(Snapshot *sn, RgbImage *ri) {
+static CanonImage *convert_page(Snapshot *sn) {

   CanonImage *im;
 
   fwrite_ppmraw(screenshot_file, sn);
   CanonImage *im;
 
   fwrite_ppmraw(screenshot_file, sn);

-  if (ri) memcpy(ri->data, sn->data, ri->h * ri->w * 3);

 
   unsigned char *pixel= sn->data;
   CANONICALISE_IMAGE(im, sn->w, sn->h, {
 
   unsigned char *pixel= sn->data;
   CANONICALISE_IMAGE(im, sn->w, sn->h, {


@@ -523,7 +521,7 @@ static void prepare_ypp_client(void) {
   raise_and_get_details();
   wait_for_stability(&current,0,0, "checking current YPP client screen...");
 
   raise_and_get_details();
   wait_for_stability(&current,0,0, "checking current YPP client screen...");
 

-  test= convert_page(current, 0);
+  test= convert_page(current);

   find_structure(test, &max_relevant_y);
   check_correct_commodities();
   Rect sunshine= find_sunshine_widget();
   find_structure(test, &max_relevant_y);
   check_correct_commodities();
   Rect sunshine= find_sunshine_widget();


@@ -553,7 +551,6 @@ static void prepare_ypp_client(void) {
 
 void take_screenshots(void) {
   Snapshot *current=0, *last=0;
 
 void take_screenshots(void) {
   Snapshot *current=0, *last=0;

-  RgbImage *page0_store;

 
   prepare_ypp_client();
   
 
   prepare_ypp_client();
   


@@ -562,8 +559,6 @@ void take_screenshots(void) {
   wait_for_stability(&current,0, send_pgup_many,
                     "paging up to top of commodity list...");
 
   wait_for_stability(&current,0, send_pgup_many,
                     "paging up to top of commodity list...");
 

-  page0_rgbimage= page0_store= alloc_rgb_image(current->w, current->h);
-

   /* now to actually page down */
   for (;;) {
     debugf("paging page %d\n",npages);
   /* now to actually page down */
   for (;;) {
     debugf("paging page %d\n",npages);


@@ -572,8 +567,8 @@ void take_screenshots(void) {
       fatal("Paging down seems to generate too many pages - max is %d.",
            MAX_PAGES);
     
       fatal("Paging down seems to generate too many pages - max is %d.",
            MAX_PAGES);
     

-    page_images[npages]= convert_page(current, page0_store);
-    free_snapshot(&last); last=current; current=0; page0_store=0;
+    page_images[npages]= convert_page(current);
+    free_snapshot(&last); last=current; current=0;

 
     debugf("PAGING page %d converted\n",npages);
 
 
     debugf("PAGING page %d converted\n",npages);
 


@@ -595,7 +590,6 @@ void take_screenshots(void) {
 
   debugf("PAGING all done.\n");
   progress_log("collected %d screenshots.",npages);
 
   debugf("PAGING all done.\n");
   progress_log("collected %d screenshots.",npages);

-  assert(!page0_store);

 }    
 
 void take_one_screenshot(void) {
 }    
 
 void take_one_screenshot(void) {


@@ -603,8 +597,7 @@ void take_one_screenshot(void) {
 
   prepare_ypp_client();
   wait_for_stability(&current,0,0, "taking screenshot...");
 
   prepare_ypp_client();
   wait_for_stability(&current,0,0, "taking screenshot...");

-  page0_rgbimage= alloc_rgb_image(current->w, current->h);
-  page_images[0]= convert_page(current, page0_rgbimage);
+  page_images[0]= convert_page(current);

   npages= 1;
   progress_log("collected single screenshot.");
 }
   npages= 1;
   progress_log("collected single screenshot.");
 }

diff --git a/pctb/structure.c b/pctb/structure.c
index 370892c2785b02fb26c9e2dbfe91f4a4f975c34c..bf4fcbd6399c496e0e800b3d05f5f16ad12d7129 100644 (file)
--- a/pctb/structure.c
+++ b/pctb/structure.c
@@ -378,7 +378,6 @@ static void file_read_image_ppm(FILE *f) {
   struct pam inpam;
   unsigned char rgb_buf[3];
   CanonImage *im;
   struct pam inpam;
   unsigned char rgb_buf[3];
   CanonImage *im;

-  RgbImage *ri=0;

 
   pnm_readpaminit(f, &inpam, sizeof(inpam));
   if (!(inpam.maxval == 255 &&
 
   pnm_readpaminit(f, &inpam, sizeof(inpam));
   if (!(inpam.maxval == 255 &&


@@ -386,9 +385,6 @@ static void file_read_image_ppm(FILE *f) {
        inpam.format == RPPM_FORMAT))
     fatal("PNM screenshot(s) file must be 8bpp 1 byte per sample RGB");
 
        inpam.format == RPPM_FORMAT))
     fatal("PNM screenshot(s) file must be 8bpp 1 byte per sample RGB");
 

-  if (!npages)
-    page0_rgbimage= ri= alloc_rgb_image(inpam.width, inpam.height);
-

   CANONICALISE_IMAGE(im, inpam.width, inpam.height, {
     int r= fread(&rgb_buf,1,3,f);
     sysassert(!ferror(f));
   CANONICALISE_IMAGE(im, inpam.width, inpam.height, {
     int r= fread(&rgb_buf,1,3,f);
     sysassert(!ferror(f));


@@ -399,8 +395,6 @@ static void file_read_image_ppm(FILE *f) {
        ((unsigned long)rgb_buf[1]<<8) |
                       (rgb_buf[2]);
 
        ((unsigned long)rgb_buf[1]<<8) |
                       (rgb_buf[2]);
 

-    if (ri)
-      CANONIMG_ALSO_STORERGB(ri);

   });
 
   sysassert(!ferror(screenshot_file));
   });
 
   sysassert(!ferror(screenshot_file));


@@ -563,7 +557,7 @@ void find_islandname(RgbImage *ri) {
 
   const unsigned char *srcp;
   unsigned char *destp, *endp;
 
   const unsigned char *srcp;
   unsigned char *destp, *endp;

-  for (srcp=page0_rgbimage->data, destp=ri->data,
+  for (srcp=page_images[0]->rgb->data, destp=ri->data,

         endp= ri->data + 3 * ri->w * ri->h;
        destp < endp;
        srcp++, destp++) {
         endp= ri->data + 3 * ri->w * ri->h;
        destp < endp;
        srcp++, destp++) {

diff --git a/pctb/structure.h b/pctb/structure.h
index ec399a2aaa7cb974dfee2b636971ad2abb348415..f3fbb72ed2d157ea01a3b103e2d2854544bb2f85 100644 (file)
--- a/pctb/structure.h
+++ b/pctb/structure.h
@@ -42,45 +42,47 @@ extern const CanonColourInfo canoncolourinfos[];
 
 CanonImage *alloc_canon_image(int w, int h);
 
 
 CanonImage *alloc_canon_image(int w, int h);
 

-#define CANONICALISE_IMAGE(im,w,h, COMPUTE_RGB) do{                    \
-    /* compute_rgb should be a number of statements, or                        \
-     * a block, which assigns to                                       \
-     *   unsigned long rgb;                                            \
-     * given the values of                                             \
-     *   int x,y;                                                      \
-     * all of which are anamorphic.  Result is stored in im.           \
-     * The COMPUTE_RGB is executed exactly once for                    \
-     * each pixel in reading order.                                    \
-     */                                                                        \
-    (im)= alloc_canon_image((w), (h));                                 \
-                                                                       \
-    int x,y;                                                           \
-    unsigned long last_rgb= ~0UL;                                      \
-    int last_c= -1;                                                    \
-    for (y=0; y<(h); y++) {                                            \
-      for (x=0; x<(w); x++) {                                          \
-        const CanonColourInfo *cci;                                    \
-        unsigned long rgb;                                             \
-       COMPUTE_RGB;                                                    \
-       if (rgb == last_rgb) {                                          \
-         (im)->d[y*(w) + x]= last_c;                                   \
-       } else {                                                        \
-         for (cci=canoncolourinfos; cci->c; cci++) {                   \
-           if (cci->rgb == rgb) {                                      \
-             last_rgb= rgb;                                            \
-             (im)->d[y*(w) + x]= last_c= cci->c;                       \
-             break;                                                    \
-           }                                                           \
-         }                                                             \
-       }                                                               \
-      }                                                                        \
-      if (DEBUGP(rect)) {                                              \
-       fprintf(debug, "%4d ",y);                                       \
-       fwrite(im->d + y*w, 1,w, debug);                                \
-       fputc('\n',debug);                                              \
-      }                                                                        \
-    }                                                                  \
-    debug_flush();                                                     \
+#define CANONICALISE_IMAGE(im,w,h, COMPUTE_RGB) do{            \
+    /* compute_rgb should be a number of statements, or                \
+     * a block, which assigns to                               \
+     *   unsigned long rgb;                                    \
+     * given the values of                                     \
+     *   int x,y;                                              \
+     * all of which are anamorphic.  Result is stored in im.   \
+     * The COMPUTE_RGB is executed exactly once for            \
+     * each pixel in reading order.                            \
+     */                                                                \
+    (im)= alloc_canon_image((w), (h));                         \
+    (im)->rgb= alloc_rgb_image((w), (h));                      \
+                                                               \
+    int x,y;                                                   \
+    unsigned long last_rgb= ~0UL;                              \
+    int last_c= -1;                                            \
+    for (y=0; y<(h); y++) {                                    \
+      for (x=0; x<(w); x++) {                                  \
+        const CanonColourInfo *cci;                            \
+        unsigned long rgb;                                     \
+       COMPUTE_RGB;                                            \
+        CANONIMG_ALSO_STORERGB((im)->rgb);                     \
+       if (rgb == last_rgb) {                                  \
+         (im)->d[y*(w) + x]= last_c;                           \
+       } else {                                                \
+         for (cci=canoncolourinfos; cci->c; cci++) {           \
+           if (cci->rgb == rgb) {                              \
+             last_rgb= rgb;                                    \
+             (im)->d[y*(w) + x]= last_c= cci->c;               \
+             break;                                            \
+           }                                                   \
+         }                                                     \
+       }                                                       \
+      }                                                                \
+      if (DEBUGP(rect)) {                                      \
+       fprintf(debug, "%4d ",y);                               \
+       fwrite(im->d + y*(w), 1,(w), debug);                    \
+       fputc('\n',debug);                                      \
+      }                                                                \
+    }                                                          \
+    debug_flush();                                             \

   }while(0)
 
 
   }while(0)