[Groff-commit] groff ./ChangeLog font/devps/prologue.ps

[Werner LEMBERG]

CVSROOT:        /cvsroot/groff
Module name:    groff
Branch:         
Changes by:     Werner LEMBERG <address@hidden> 05/06/21 05:54:05

Modified files:
        .              : ChangeLog 
        font/devps     : prologue.ps 

Log message:
        * font/devps/prologue.ps (EP): Dont' use `bind'.  Reason: Using
        `bind' to define a procedure which contains `showpage' means that an
        invocation of `showpage' may execute wrongly (the exact effect
        depending on the interpreter).  Not usually a problem except under
        transformations, such as combining pages.
        (PEND): Don't call `clear'.  Reason: Using `clear' in the definition
        of /PEND means that stack underflow will occur if the stack is
        deliberately non-empty before document inclusion.  If included
        files are found to be generally badly behaved, we could replace the
        `clear' with `mark' in /PBEGIN and `cleartomark' in /PEND (I've
        never found this necessary).

CVSWeb URLs:
http://savannah.gnu.org/cgi-bin/viewcvs/groff/groff/ChangeLog.diff?tr1=1.834&tr2=1.835&r1=text&r2=text
http://savannah.gnu.org/cgi-bin/viewcvs/groff/groff/font/devps/prologue.ps.diff?tr1=1.5&tr2=1.6&r1=text&r2=text

Patches:
Index: groff/ChangeLog
diff -u groff/ChangeLog:1.834 groff/ChangeLog:1.835
--- groff/ChangeLog:1.834       Mon Jun 20 21:33:43 2005
+++ groff/ChangeLog     Tue Jun 21 05:54:04 2005
@@ -1,3 +1,17 @@
+2005-06-20  Denis M, Wilson  address@hidden
+
+       * font/devps/prologue.ps (EP): Dont' use `bind'.  Reason: Using
+       `bind' to define a procedure which contains `showpage' means that an
+       invocation of `showpage' may execute wrongly (the exact effect
+       depending on the interpreter).  Not usually a problem except under
+       transformations, such as combining pages.
+       (PEND): Don't call `clear'.  Reason: Using `clear' in the definition
+       of /PEND means that stack underflow will occur if the stack is
+       deliberately non-empty before document inclusion.  If included
+       files are found to be generally badly behaved, we could replace the
+       `clear' with `mark' in /PBEGIN and `cleartomark' in /PEND (I've
+       never found this necessary).
+
 2005-06-20  Jörgen Grahn  <address@hidden>
 
        * src/preproc/soelim/soelim.cpp (do_file): Append trailing zero
Index: groff/font/devps/prologue.ps
diff -u groff/font/devps/prologue.ps:1.5 groff/font/devps/prologue.ps:1.6
--- groff/font/devps/prologue.ps:1.5    Thu Feb 27 15:55:19 2003
+++ groff/font/devps/prologue.ps        Tue Jun 21 05:54:05 2005
@@ -91,7 +91,7 @@
 /EP {
        level0 restore
        showpage
-} bind def
+} def
 
 
 % centerx centery radius startangle endangle DA -
@@ -243,7 +243,6 @@
 } bind def
 
 /PEND {
-       clear
        countdictstack CNT sub { end } repeat
        level1 restore
 } bind def