X-Git-Url: http://www.chiark.greenend.org.uk/ucgi/~yarrgweb/git?a=blobdiff_plain;f=pctb%2Fpages.c;h=dfa9108537bbf7b5a9c19d507a443f6caa13851c;hb=3768c30b78a0afa7ca2c69c21365429c98cd868a;hp=dae3d2138bd0d8a3ae0a874b3fc8ff74ff8562f7;hpb=4d7eb93b7c82da31f928349ac7c6cd551d78ca57;p=ypp-sc-tools.db-live.git

diff --git a/pctb/pages.c b/pctb/pages.c
index dae3d21..dfa9108 100644
--- a/pctb/pages.c
+++ b/pctb/pages.c
@@ -1,5 +1,34 @@
 /*
-  */
+ * Interaction with the YPP client via X11
+ */
+/*
+ *  This is part of ypp-sc-tools, a set of third-party tools for assisting
+ *  players of Yohoho Puzzle Pirates.
+ * 
+ *  Copyright (C) 2009 Ian Jackson <ijackson@chiark.greenend.org.uk>
+ * 
+ *  This program is free software: you can redistribute it and/or modify
+ *  it under the terms of the GNU General Public License as published by
+ *  the Free Software Foundation, either version 3 of the License, or
+ *  (at your option) any later version.
+ * 
+ *  This program is distributed in the hope that it will be useful,
+ *  but WITHOUT ANY WARRANTY; without even the implied warranty of
+ *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ *  GNU General Public License for more details.
+ * 
+ *  You should have received a copy of the GNU General Public License
+ *  along with this program.  If not, see <http://www.gnu.org/licenses/>.
+ * 
+ *  Yohoho and Puzzle Pirates are probably trademarks of Three Rings and
+ *  are used without permission.  This program is not endorsed or
+ *  sponsored by Three Rings.
+ */
+
+/*
+ * Only this file #includes the X11 headers, as they're quite
+ * pollutant of the namespace.
+ */
 
 #include "structure.h"
 
@@ -10,6 +39,7 @@
 
 CanonImage *page_images[MAX_PAGES];
 int npages;
+RgbImage *page0_rgbimage;
 
 char *ocean, *pirate;
 
@@ -18,18 +48,24 @@ static Window id;
 static Display *disp;
 static struct timeval tv_startup;
 static unsigned wwidth, wheight;
+static int max_relevant_y= -1;
 
 DEBUG_DEFINE_DEBUGF(pages)
 
+#define xassert(what)					\
+  ((what) ? (void)0 :					\
+   fatal("X11 operation unexpectedly failed."		\
+	 " %s:%d: %s\n", __FILE__,__LINE__,#what))
+
 static KeyCode keycode(KeySym sym) {
   return XKeysymToKeycode(disp,sym);
 }
 
 void screenshot_startup(void) {
-  int r;
   progress("starting...");
-  disp= XOpenDisplay(0);  eassert(disp);
-  r= gettimeofday(&tv_startup,0);  eassert(!r);
+  disp= XOpenDisplay(0);
+  if (!disp) fatal("Unable to open X11 display.");
+  sysassert(! gettimeofday(&tv_startup,0) );
 }
 
 /*---------- pager ----------*/
@@ -41,49 +77,46 @@ static const double min_update_allowance= 0.25;
 
 static double timestamp(void) {
   struct timeval tv;
-  int r;
   
-  r= gettimeofday(&tv,0);  eassert(!r);
+  sysassert(! gettimeofday(&tv,0) );
   double t= (tv.tv_sec - tv_startup.tv_sec) +
             (tv.tv_usec - tv_startup.tv_usec) * 1e-6;
   debugf("PAGING %f\n",t);
   return t;
 }
 static void delay(double need_sleep) {
-  int r;
   debugf("PAGING     delay %f\n",need_sleep);
-  r= usleep(need_sleep * 1e6);  eassert(!r);
+  sysassert(! usleep(need_sleep * 1e6) );
 }
 
 static void sync_after_input(void) {
-  int r;
-  r= XSync(disp, False);  eassert(r);
+  xassert( XSync(disp, False) );
   last_input= timestamp();
 }
 
 static void translate_coords_toroot(int wx, int wy, int *rx, int *ry) {
-  int r;
   Window dummy;
-  r= XTranslateCoordinates(disp, id,attr.root, wx,wy, rx,ry, &dummy);
-  eassert(r);
+  xassert( XTranslateCoordinates(disp, id,attr.root, wx,wy, rx,ry, &dummy) );
 }
 
 static void check_client_window_all_on_screen(void) {
-  int r;
-  int rxpos, rypos;
+  Rect onroot;
   unsigned rwidth, rheight;
   Window dummy;
   unsigned bd, depth;
+  int rxpos, rypos;
 
-  r= XGetGeometry(disp,attr.root, &dummy, &rxpos,&rypos,
-		  &rwidth, &rheight,
-		  &bd,&depth);
+  xassert( XGetGeometry(disp,attr.root, &dummy, &rxpos,&rypos,
+			&rwidth, &rheight,
+			&bd,&depth) );
   
-  translate_coords_toroot(0,0, &rxpos,&rypos);
-  eassert(rxpos>=0 && rypos>=0);
-
-  translate_coords_toroot(wwidth-1,wheight-1, &rxpos,&rypos);
-  eassert(rxpos<rwidth && rypos<rheight);
+  translate_coords_toroot(0,0, &onroot.tl.x,&onroot.tl.y);
+  translate_coords_toroot(wwidth-1,wheight-1, &onroot.br.x,&onroot.br.y);
+  if (!(onroot.tl.x >= 0 &&
+	onroot.tl.y >= 0 &&
+	onroot.br.x < rwidth &&
+	onroot.br.y < rheight))
+    fatal("YPP client window is not entirely on the screen.");
 }
 
 static void check_not_disturbed(void) {
@@ -96,40 +129,68 @@ static void check_not_disturbed(void) {
 
     switch (ev.type) {
     case VisibilityNotify:
-      eassert(ev.xvisibility.state == VisibilityUnobscured);
+      if (ev.xvisibility.state != VisibilityUnobscured)
+	fatal("YPP client window has become obscured.");
       break;
     case ConfigureNotify:
       check_client_window_all_on_screen();
       break;
     case FocusOut:
-      eassert(!"focus left YPP client window");
+      fatal("Focus left YPP client window.");
       break;
     case FocusIn:
       warning("focus entered YPP client window ?!");
       break;
     default:
-      eassert(!"unexpected X11 event");
+      fatal("Received unexpected X11 event (type code %d)!", ev.type);
     }
   }
 }      
 
 static void send_key(KeySym sym) {
+  check_not_disturbed();
   XTestFakeKeyEvent(disp, keycode(sym),1, 10);
   XTestFakeKeyEvent(disp, keycode(sym),0, 10);
 }
+static void mouse_1_updown_here(void) {
+  check_not_disturbed();
+  XTestFakeButtonEvent(disp,1,1, 50);
+  XTestFakeButtonEvent(disp,1,0, 50);
+}
+static void mouse_1_updown(int x, int y) {
+  check_not_disturbed();
+  int screen= XScreenNumberOfScreen(attr.screen);
+  int xpos, ypos;
+  translate_coords_toroot(x,y, &xpos,&ypos);
+  XTestFakeMotionEvent(disp, screen, xpos,ypos, 0);
+  mouse_1_updown_here();
+}
+
+static int pgupdown;
 
 static void send_pgup_many(void) {
   int i;
-  for (i=0; i<25; i++)
+  for (i=0; i<25; i++) {
     send_key(XK_Prior);
+    pgupdown--;
+  }
   debugf("PAGING   PageUp x %d\n",i);
   sync_after_input();
 }
 static void send_pgdown(void) {
   send_key(XK_Next);
+  pgupdown++;
   debugf("PAGING   PageDown\n");
   sync_after_input();
 }
+static void send_pgdown_torestore(void) {
+  debugf("PAGING   PageDown x %d\n", -pgupdown);
+  while (pgupdown < 0) {
+    send_key(XK_Next);
+    pgupdown++;
+  }
+  sync_after_input();
+}
 
 static void free_snapshot(Snapshot **io) {
   if (*io) XDestroyImage(*io);
@@ -156,7 +217,12 @@ static int identical(const Snapshot *a, const Snapshot *b) {
 	a->bytes_per_line == b->bytes_per_line &&
 	a->format == b->format))
     return 0;
-  return !memcmp(a->data, b->data, a->bytes_per_line * a->height);
+
+  int compare_to= a->height;
+  if (max_relevant_y && compare_to > max_relevant_y)
+    compare_to= max_relevant_y;
+  
+  return !memcmp(a->data, b->data, a->bytes_per_line * compare_to);
 }
 
 static void wait_for_stability(Snapshot **output,
@@ -173,7 +239,6 @@ static void wait_for_stability(Snapshot **output,
   va_start(al,fmt);
 
   Snapshot *last=0;
-  int r;
   /* waits longer if we're going to return an image identical to previously
    * if previously==0, all images are considered identical to it */
 
@@ -182,7 +247,7 @@ static void wait_for_stability(Snapshot **output,
 	  last_input, previously);
 
   char *doing;
-  r= vasprintf(&doing,fmt,al);  eassert(r>=0);
+  sysassert( vasprintf(&doing,fmt,al) >=0);
 
   progress("%s",doing);
 
@@ -223,7 +288,6 @@ static void wait_for_stability(Snapshot **output,
 }
 
 static void raise_and_get_details(void) {
-  int r;
   int evbase,errbase,majver,minver;
   int wxpos, wypos;
   unsigned bd,depth;
@@ -232,56 +296,51 @@ static void raise_and_get_details(void) {
 
   debugf("PAGING raise_and_get_details\n");
 
-  r= XTestQueryExtension(disp, &evbase,&errbase,&majver,&minver);
-  eassert(r==True);
+  int xtest= XTestQueryExtension(disp, &evbase,&errbase,&majver,&minver);
+  if (!xtest) fatal("X server does not support the XTEST extension.");
 
-  r= XRaiseWindow(disp, id);  eassert(r);
+  xassert( XRaiseWindow(disp, id) );
   /* in case VisibilityNotify triggers right away before we have had a
    * change to raise; to avoid falsely detecting lowering in that case */
   
-  r= XSelectInput(disp, id,
-		  StructureNotifyMask|
-		  VisibilityChangeMask
-		  );  eassert(r);
+  xassert( XSelectInput(disp, id,
+			StructureNotifyMask|
+			VisibilityChangeMask
+			) );
 
-  r= XRaiseWindow(disp, id);  eassert(r);
+  xassert( XRaiseWindow(disp, id) );
   /* in case the window was lowered between our Raise and our SelectInput;
    * to avoid failing to detect that lowering */
 
-  r= XGetWindowAttributes(disp, id, &attr);  eassert(r);
-  r= XGetGeometry(disp,id, &attr.root,
-		  &wxpos,&wypos, &wwidth,&wheight,
-		  &bd,&depth);
+  xassert( XGetWindowAttributes(disp, id, &attr) );
+  xassert( XGetGeometry(disp,id, &attr.root,
+			&wxpos,&wypos, &wwidth,&wheight,
+			&bd,&depth) );
 
-  eassert(wwidth >= 320 && wheight >= 320);
+  if (!(wwidth >= 320 && wheight >= 320))
+    fatal("YPP client window is implausibly small?");
 
   check_client_window_all_on_screen();
 }
 
-static void set_focus(void) {
-  int r;
+static void set_focus_commodity(void) {
   int screen= XScreenNumberOfScreen(attr.screen);
 
   progress("taking control of YPP client window...");
 
   debugf("PAGING set_focus\n");
 
-  int xpos, ypos;
-  translate_coords_toroot(160,160, &xpos,&ypos);
-  XTestFakeMotionEvent(disp,screen, xpos,ypos, 0);
-
-  XTestFakeButtonEvent(disp,1,1, 50);
-  XTestFakeButtonEvent(disp,1,0, 50);
-
+  mouse_1_updown(160,160);
   sync_after_input();
 
   delay(0.5);
-  r= XSelectInput(disp, id,
-		  StructureNotifyMask|
-		  VisibilityChangeMask|
-		  FocusChangeMask
-		  );  eassert(r);
+  xassert( XSelectInput(disp, id,
+			StructureNotifyMask|
+			VisibilityChangeMask|
+			FocusChangeMask
+			) );
 
+  int xpos,ypos;
   translate_coords_toroot(10,10, &xpos,&ypos);
   XTestFakeMotionEvent(disp,screen, xpos,ypos, 0);
 
@@ -314,11 +373,11 @@ static void compute_shift_mask(ShMask *sm, unsigned long ximage_mask) {
   assert(sm->rshift < LONG_BIT);
 }
 
-static CanonImage *convert_page(Snapshot *sn) {
+static CanonImage *convert_page(Snapshot *sn, RgbImage *ri) {
   ShMask shiftmasks[3];
   CanonImage *im;
 
-  fprintf(screenshots_file,
+  fprintf(screenshot_file,
 	  "P6\n"
 	  "%d %d\n"
 	  "255\n", sn->width, sn->height);
@@ -339,28 +398,50 @@ static CanonImage *convert_page(Snapshot *sn) {
 	((xrgb << shiftmasks[i].lshift)
 	      >> shiftmasks[i].rshift) & SAMPLEMASK;
       rgb |= sample;
-      fputc(sample, screenshots_file);
+      fputc(sample, screenshot_file);
     }
+    if (ri)
+      CANONIMG_ALSO_STORERGB(ri);
   });
 
-  eassert(!fflush(screenshots_file));
+  sysassert(!ferror(screenshot_file));
+  sysassert(!fflush(screenshot_file));
 
   return im;
 }
 
-void take_screenshots(void) {
-  Snapshot *current=0, *last=0;
+static void prepare_ypp_client(void) {
   CanonImage *test;
-
+  Snapshot *current=0;
+  
   /* find the window and check it's on the right kind of screen */
   raise_and_get_details();
   wait_for_stability(&current,0,0, "checking current YPP client screen...");
-  test= convert_page(current);
-  find_structure(test);
+  test= convert_page(current, 0);
+  find_structure(test, &max_relevant_y);
+  check_correct_commodities();
   free(test);
+  free_snapshot(&current);
 
+  progress("requesting status information...");
+  mouse_1_updown(250, wheight-10);
+  mouse_1_updown_here();
+  mouse_1_updown_here();
+  XSync(disp,False);
+  check_not_disturbed();
+  send_key(XK_slash);
+  send_key(XK_w);
+  send_key(XK_Return);
+  sync_after_input();
+}
+
+void take_screenshots(void) {
+  Snapshot *current=0, *last=0;
+
+  prepare_ypp_client();
+  
   /* page to the top - keep pressing page up until the image stops changing */
-  set_focus();
+  set_focus_commodity();
   wait_for_stability(&current,0, send_pgup_many,
 		     "paging up to top of commodity list...");
 
@@ -368,8 +449,11 @@ void take_screenshots(void) {
   for (;;) {
     debugf("paging page %d\n",npages);
 
-    eassert(npages < MAX_PAGES);
-    page_images[npages]= convert_page(current);
+    if (!(npages < MAX_PAGES))
+      fatal("Paging down seems to generate too many pages - max is %d.",
+	    MAX_PAGES);
+    
+    page_images[npages]= convert_page(current, 0);
     free_snapshot(&last); last=current; current=0;
 
     debugf("PAGING page %d converted\n",npages);
@@ -387,17 +471,20 @@ void take_screenshots(void) {
     send_pgdown();
     npages++;
   }
+  progress("finishing with the YPP client...");
+  send_pgdown_torestore();
+
   debugf("PAGING all done.\n");
   progress_log("collected %d screenshots.",npages);
 }    
 
 void take_one_screenshot(void) {
   Snapshot *current=0;
-  
-  raise_and_get_details();
-  sync_after_input();
+
+  prepare_ypp_client();
   wait_for_stability(&current,0,0, "taking screenshot...");
-  page_images[0]= convert_page(current);
+  page0_rgbimage= alloc_rgb_image(current->width, current->height);
+  page_images[0]= convert_page(current, 0);
   npages= 1;
   progress_log("collected single screenshot.");
 }
@@ -410,7 +497,7 @@ DEBUG_DEFINE_SOME_DEBUGF(findypp,debugfind)
 
 void find_yppclient_window(void) {
   Window root, gotroot, gotparent;
-  int screen, r;
+  int screen;
   int nfound=0;
   
   if (id) return;
@@ -423,7 +510,7 @@ void find_yppclient_window(void) {
 #define S(x) (sizeof((x))-1)
 
   Atom wm_name= XInternAtom(disp,"WM_NAME",True);
-  eassert(wm_name != None);
+  xassert(wm_name != None);
 
   for (screen=0; screen<ScreenCount(disp); screen++) {
     debugfind("FINDYPP screen %d\n", screen);
@@ -431,10 +518,9 @@ void find_yppclient_window(void) {
     unsigned int nchildren1;
     Window *children1=0;
 
-    r= XQueryTree(disp,root,
+    xassert( XQueryTree(disp,root,
 		  &gotroot,&gotparent,
-		  &children1,&nchildren1);
-    eassert(r);
+		  &children1,&nchildren1) );
     debugfind("FINDYPP screen %d nchildren1=%d\n", screen, nchildren1);
 
     int i;
@@ -443,10 +529,9 @@ void find_yppclient_window(void) {
       unsigned int nchildren2;
       Window *children2=0;
 
-      r= XQueryTree(disp,w1,
-		    &gotroot,&gotparent,
-		    &children2,&nchildren2);
-      eassert(r);
+      xassert( XQueryTree(disp,w1,
+			  &gotroot,&gotparent,
+			  &children2,&nchildren2) );
       debugfind("FINDYPP screen %d c1[%2d]=0x%08lx nchildren2=%d\n",
 		screen, i, (unsigned long)w1, nchildren2);
 
@@ -461,10 +546,9 @@ void find_yppclient_window(void) {
 	unsigned long len, gotbytesafter;
 	char *title;
 	unsigned char *gottitle=0;
-	r= XGetWindowProperty(disp,w2, wm_name,0,512, False,
-			      AnyPropertyType,&gottype, &gotfmt, &len,
-			      &gotbytesafter, &gottitle);
-	eassert(!r);
+	xassert( !XGetWindowProperty(disp,w2, wm_name,0,512, False,
+				     AnyPropertyType,&gottype, &gotfmt, &len,
+				     &gotbytesafter, &gottitle) );
 	title= (char*)gottitle;
 
 	if (DEBUGP(findypp)) {
@@ -498,8 +582,8 @@ void find_yppclient_window(void) {
 	REQUIRE( !memcmp(title + len - S(suffix), suffix, S(suffix))  );
 	REQUIRE( !memcmp(spc1,                    onthe,  S(onthe))  );
 
-#define ASSIGN(what, start, end) do {					 \
-	r= asprintf(&what, "%.*s", (end)-(start), start);  eassert(r>0); \
+#define ASSIGN(what, start, end) do {					\
+	sysassert( asprintf(&what, "%.*s", (end)-(start), start) >0 );	\
      }while(0)
 	ASSIGN(pirate, title + S(prefix),  spc1);
 	ASSIGN(ocean,  spc1 + S(onthe),   (title + len) - S(suffix));
@@ -507,7 +591,7 @@ void find_yppclient_window(void) {
 	debugfind(" YES!\n");
 	id= w2;
 	nfound++;
-	progress_log("found YPP client [--window-id 0x%lx]:"
+	progress_log("found YPP client (0x%lx):"
 		     " %s ocean - %s.",
 		     (unsigned long)id, ocean, pirate);
       }
@@ -515,5 +599,10 @@ void find_yppclient_window(void) {
     }
     if (children1) XFree(children1);
   }
-  eassert(nfound==1);
+  if (nfound>1)
+    fatal("Found several YPP clients."
+	  " Close one, or specify the windowid with --window-id.\n");
+  if (nfound<1)
+    fatal("Did not find YPP client."
+	  " Use --window-id and/or report this as a fault.\n");
 }