[gs-cvs] rev 7806 - in trunk/gs: doc src

[alexcher at ghostscript.com]

Author: alexcher
Date: 2007-03-27 10:39:51 -0700 (Tue, 27 Mar 2007)
New Revision: 7806

Modified:
   trunk/gs/doc/pscet_status.txt
   trunk/gs/src/zchar.c
Log:
Make FontBBox a required attribute in CPSI compatibility mode.
PLRM says that it's a required attribute and recent Adobe interpreters
enforce this. Fix CET 15-11.

DIFFERENCES:
No other CET differences. Comparefiles don't run in the compatibility mode.


Modified: trunk/gs/doc/pscet_status.txt
===================================================================
--- trunk/gs/doc/pscet_status.txt	2007-03-27 02:55:14 UTC (rev 7805)
+++ trunk/gs/doc/pscet_status.txt	2007-03-27 17:39:51 UTC (rev 7806)
@@ -1920,7 +1920,9 @@
 		assign: Ray.
 
 11-20-6  DIFF	In special test B, Adobe has rangechecks, we don't. In
-	 special test A, we have one rangecheck they don't. - assign Raph
+		special test A, we have one rangecheck they don't.
+                The test is about interaction of setbbox with save,
+                gsave and other operators. - assign Raph
 
 11-20-7  OK	
 
@@ -3491,8 +3493,7 @@
 
 15-10-4  OK	Minor rasterizing differences. -Tor
 
-15-11-1  DIFF	Top row of circles is visible in GS and empty in CPSI. (/NotDefined null BuildFont) The page says Device Dependent behavior.
-                Assign: Ray
+15-11-1  OK  	Fixed by rev. 7806. - Alex
 
 15-14-1  OK	Minor differences visually reviewed by RJJ
 

Modified: trunk/gs/src/zchar.c
===================================================================
--- trunk/gs/src/zchar.c	2007-03-27 02:55:14 UTC (rev 7805)
+++ trunk/gs/src/zchar.c	2007-03-27 17:39:51 UTC (rev 7806)
@@ -39,6 +39,8 @@
 #include "store.h"
 #include "zchar42.h"
 
+extern bool CPSI_mode;
+
 /* Forward references */
 private bool map_glyph_to_char(const gs_memory_t *mem, 
 			       const ref *, const ref *, ref *);
@@ -853,7 +855,7 @@
 
     /*
      * Pre-clear the bbox in case it's invalid.  The Red Books say that
-     * FontBBox is required, but the Adobe interpreters don't require
+     * FontBBox is required, but old Adobe interpreters don't require
      * it, and a few user-written fonts don't supply it, or supply one
      * of the wrong size (!); also, PageMaker 5.0 (an Adobe product!)
      * sometimes emits an absurd bbox for Type 1 fonts converted from
@@ -885,6 +887,8 @@
 		)
 		bbox[0] = bbox[1] = bbox[2] = bbox[3] = 0.0;
 	}
+    } else if (CPSI_mode) {
+        return_error(e_invalidfont); /* CPSI requires FontBBox */
     }
     return 0;
 }