Attachment 20848 Details for Bug 18908 – a patch fixes this problem.

[patch] a patch fixes this problem.

pdftops-large-image.patch (text/plain), 4.60 KB, created by Koji Otani on 2008-12-05 19:54:05 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Koji Otani

Created: 2008-12-05 19:54:05 UTC

Size: 4.60 KB

patch

obsolete

>diff --git a/poppler/PSOutputDev.cc b/poppler/PSOutputDev.cc
>index 014036b..75d7ba7 100644
>--- a/poppler/PSOutputDev.cc
>+++ b/poppler/PSOutputDev.cc
>@@ -2610,6 +2610,7 @@ void PSOutputDev::setupImage(Ref id, Stream *str) {
>   GooString *s;
>   int c;
>   int size, line, col, i;
>+  int outerSize, outer;
> 
>   // check if image is already setup
>   for (i = 0; i < imgIDLen; ++i) {
>@@ -2696,56 +2697,72 @@ void PSOutputDev::setupImage(Ref id, Stream *str) {
>   if (useRLE) {
>     ++size;
>   }
>+  outerSize = size/65535 + 1;
>+
>   writePSFmt("{0:d} array dup /ImData_{1:d}_{2:d} exch def\n",
>-	     size, id.num, id.gen);
>+	     outerSize, id.num, id.gen);
>   str->close();
> 
>   // write the data into the array
>   str->reset();
>-  line = col = 0;
>-  writePS((char *)(useASCIIHex ? "dup 0 <" : "dup 0 <~"));
>-  do {
>-    do {
>-      c = str->getChar();
>-    } while (c == '\n' || c == '\r');
>-    if (c == (useASCIIHex ? '>' : '~') || c == EOF) {
>-      break;
>-    }
>-    if (c == 'z') {
>-      writePSChar(c);
>-      ++col;
>-    } else {
>-      writePSChar(c);
>-      ++col;
>-      for (i = 1; i <= (useASCIIHex ? 1 : 4); ++i) {
>-	do {
>-	  c = str->getChar();
>-	} while (c == '\n' || c == '\r');
>-	if (c == (useASCIIHex ? '>' : '~') || c == EOF) {
>-	  break;
>-	}
>+  for (outer = 0;outer < outerSize;outer++) {
>+    int innerSize = size > 65535 ? 65535 : size;
>+
>+    // put the inner array into the outer array
>+    writePSFmt("{0:d} array 1 index {1:d} 2 index put\n",
>+	       innerSize, outer);
>+    line = col = 0;
>+    writePS((char *)(useASCIIHex ? "dup 0 <" : "dup 0 <~"));
>+    for (;;) {
>+      do {
>+	c = str->getChar();
>+      } while (c == '\n' || c == '\r');
>+      if (c == (useASCIIHex ? '>' : '~') || c == EOF) {
>+	break;
>+      }
>+      if (c == 'z') {
>+	writePSChar(c);
>+	++col;
>+      } else {
> 	writePSChar(c);
> 	++col;
>+	for (i = 1; i <= (useASCIIHex ? 1 : 4); ++i) {
>+	  do {
>+	    c = str->getChar();
>+	  } while (c == '\n' || c == '\r');
>+	  if (c == (useASCIIHex ? '>' : '~') || c == EOF) {
>+	    break;
>+	  }
>+	  writePSChar(c);
>+	  ++col;
>+	}
>+      }
>+      // each line is: "dup nnnnn <~...data...~> put<eol>"
>+      // so max data length = 255 - 20 = 235
>+      // chunks are 1 or 4 bytes each, so we have to stop at 232
>+      // but make it 225 just to be safe
>+      if (col > 225) {
>+	writePS((char *)(useASCIIHex ? "> put\n" : "~> put\n"));
>+	++line;
>+	if (line >= innerSize) break;
>+	writePSFmt((char *)(useASCIIHex ? "dup {0:d} <" : "dup {0:d} <~"), line);
>+	col = 0;
>       }
>     }
>-    // each line is: "dup nnnnn <~...data...~> put<eol>"
>-    // so max data length = 255 - 20 = 235
>-    // chunks are 1 or 4 bytes each, so we have to stop at 232
>-    // but make it 225 just to be safe
>-    if (col > 225) {
>+    if (c == (useASCIIHex ? '>' : '~') || c == EOF) {
>       writePS((char *)(useASCIIHex ? "> put\n" : "~> put\n"));
>-      ++line;
>-      writePSFmt((char *)(useASCIIHex ? "dup {0:d} <" : "dup {0:d} <~"), line);
>-      col = 0;
>+      if (useRLE) {
>+	++line;
>+	writePSFmt("{0:d} <> put\n", line);
>+      } else {
>+	writePS("pop\n");
>+      }
>+      break;
>     }
>-  } while (c != (useASCIIHex ? '>' : '~') && c != EOF);
>-  writePS((char *)(useASCIIHex ? "> put\n" : "~> put\n"));
>-  if (useRLE) {
>-    ++line;
>-    writePSFmt("{0:d} <> put\n", line);
>-  } else {
>     writePS("pop\n");
>+    size -= innerSize;
>   }
>+  writePS("pop\n");
>   str->close();
> 
>   delete str;
>@@ -4845,7 +4862,7 @@ void PSOutputDev::doImageL2(Object *ref, GfxImageColorMap *colorMap,
>       // make sure the image is setup, it sometimes is not like on bug #17645
>       setupImage(ref->getRef(), str);
>       // set up to use the array already created by setupImages()
>-      writePSFmt("ImData_{0:d}_{1:d} 0\n", ref->getRefNum(), ref->getRefGen());
>+      writePSFmt("ImData_{0:d}_{1:d} 0 0\n",ref->getRefNum(), ref->getRefGen());
>     }
>   }
> 
>@@ -4899,7 +4916,8 @@ void PSOutputDev::doImageL2(Object *ref, GfxImageColorMap *colorMap,
> 
>   // data source
>   if (mode == psModeForm || inType3Char || preload) {
>-    writePS("  /DataSource { 2 copy get exch 1 add exch }\n");
>+    writePS("  /DataSource { dup 65535 ge { pop 1 add 0 } if 2 index 2"
>+      " index get 1 index get exch 1 add exch }\n");
>   } else {
>     writePS("  /DataSource currentfile\n");
>   }
>@@ -5186,7 +5204,8 @@ void PSOutputDev::doImageL3(Object *ref, GfxImageColorMap *colorMap,
> 
>   // data source
>   if (mode == psModeForm || inType3Char || preload) {
>-    writePS("  /DataSource { 2 copy get exch 1 add exch }\n");
>+    writePS("  /DataSource { dup 65535 ge { pop 1 add 0 } if 2 index 2"
>+      " index get 1 index get exch 1 add exch }\n");
>   } else {
>     writePS("  /DataSource currentfile\n");
>   }

Actions: View | Splinter Review

Attachments on bug 18908: 20848 | 20885 | 20972 | 27628 | 27707

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.