Attachment 49356 Details for Bug 39361 – Fix and optimization

[patch] Fix and optimization

poppler.patch (text/plain), 3.20 KB, created by Daniel Glöckner on 2011-07-20 15:38:16 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Daniel Glöckner

Created: 2011-07-20 15:38:16 UTC

Size: 3.20 KB

patch

obsolete

>diff --git a/poppler/JPEG2000Stream.cc b/poppler/JPEG2000Stream.cc
>index d52f088..ed8651a 100644
>--- a/poppler/JPEG2000Stream.cc
>+++ b/poppler/JPEG2000Stream.cc
>@@ -17,6 +17,8 @@ JPXStream::JPXStream(Stream *strA) : FilterStream(strA)
>   inited = gFalse;
>   image = NULL;
>   dinfo = NULL;
>+  npixels = 0;
>+  ncomps = 0;
> }
> 
> JPXStream::~JPXStream() {
>@@ -26,12 +28,14 @@ JPXStream::~JPXStream() {
> 
> void JPXStream::reset() {
>   counter = 0;
>+  ccounter = 0;
> }
> 
> void JPXStream::close() {
>   if (image != NULL) {
>     opj_image_destroy(image);
>     image = NULL;
>+    npixels = 0;
>   }
>   if (dinfo != NULL) {
>     opj_destroy_decompress(dinfo);
>@@ -40,7 +44,7 @@ void JPXStream::close() {
> }
> 
> int JPXStream::getPos() {
>-  return counter;
>+  return counter * ncomps + ccounter;
> }
> 
> int JPXStream::getChars(int nChars, Guchar *buffer) {
>@@ -73,7 +77,37 @@ void JPXStream::init()
>   init2(buf, length, CODEC_JP2);
>   free(buf);
> 
>+  if (image) {
>+    npixels = image->comps[0].w * image->comps[0].h;
>+    ncomps = image->numcomps;
>+    for (int component = 0; component < ncomps; component++) {
>+      if (image->comps[component].data == NULL) {
>+        close();
>+        break;
>+      }
>+      unsigned char *cdata = (unsigned char *)image->comps[component].data;
>+      int adjust = 0;
>+      if (image->comps[component].prec > 8)
>+	adjust = image->comps[component].prec - 8;
>+      int sgndcorr = 0;
>+      if (image->comps[component].sgnd)
>+	sgndcorr = 1 << (image->comps[0].prec - 1);
>+      for (int i = 0; i < npixels; i++) {
>+	int r = image->comps[component].data[i];
>+	r += sgndcorr;
>+	if (adjust) {
>+	  r = (r >> adjust)+((r >> (adjust-1))%2);
>+	  if (unlikely(r > 255))
>+	    r = 255;
>+        }
>+	*(cdata++) = r;
>+      }
>+    }
>+  } else
>+    npixels = 0;
>+
>   counter = 0;
>+  ccounter = 0;
>   inited = gTrue;
> }
> 
>diff --git a/poppler/JPEG2000Stream.h b/poppler/JPEG2000Stream.h
>index bda2721..981f889 100644
>--- a/poppler/JPEG2000Stream.h
>+++ b/poppler/JPEG2000Stream.h
>@@ -44,42 +44,27 @@ private:
> 
>   inline int doGetChar() {
>     int result = doLookChar();
>-    ++counter;
>+    if (++ccounter == ncomps) {
>+      ccounter = 0;
>+      ++counter;
>+    }
>     return result;
>   }
> 
>   inline int doLookChar() {
>-    if (inited == gFalse) init();
>-
>-    if (!image) return EOF;
>-
>-    int w = image->comps[0].w;
>-    int h = image->comps[0].h;
>-
>-    int y = (counter / image->numcomps) / w;
>-    int x = (counter / image->numcomps) % w;
>-    if (y >= h) return EOF;
>-
>-    int component = counter % image->numcomps;
>-
>-    int adjust = 0;
>-    if (image->comps[component].prec > 8) {
>-      adjust = image->comps[component].prec - 8;
>-    }
>-
>-    if (unlikely(image->comps[component].data == NULL)) return EOF;
>-
>-    int r = image->comps[component].data[y * w + x];
>-    r += (image->comps[component].sgnd ? 1 << (image->comps[0].prec - 1) : 0);
>+    if (unlikely(inited == gFalse)) init();
> 
>-    unsigned char rc = (unsigned char) ((r >> adjust)+((r >> (adjust-1))%2));
>+    if (unlikely(counter >= npixels)) return EOF;
> 
>-    return rc;
>+    return ((unsigned char *)image->comps[ccounter].data)[counter];
>   }
> 
>   opj_image_t *image;
>   opj_dinfo_t *dinfo;
>   int counter;
>+  int ccounter;
>+  int npixels;
>+  int ncomps;
>   GBool inited;
> };
>

Actions: View | Splinter Review

Attachments on bug 39361: 49280 | 49294 | 49356

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.