Attachment 88664 Details for Bug 152780

[patch] proposed patch

poppler-font.patch (text/plain), 3.90 KB, created by Radek Doulik on 2006-06-12 13:11:32 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Radek Doulik

Created: 2006-06-12 13:11:32 UTC

Size: 3.90 KB

patch

obsolete

>diff -rup ../poppler-0.4.5/poppler/Gfx.cc ./poppler/Gfx.cc
>--- ../poppler-0.4.5/poppler/Gfx.cc	2005-09-04 22:34:33.000000000 +0200
>+++ ./poppler/Gfx.cc	2006-06-12 14:30:21.000000000 +0200
>@@ -2154,6 +2154,8 @@ void Gfx::opSetFont(Object args[], int n
> 	   args[1].getNum());
>     fflush(stdout);
>   }
>+
>+  font->incRefCnt();
>   state->setFont(font, args[1].getNum());
>   fontChanged = gTrue;
> }
>diff -rup ../poppler-0.4.5/poppler/GfxFont.cc ./poppler/GfxFont.cc
>--- ../poppler-0.4.5/poppler/GfxFont.cc	2005-08-16 05:34:01.000000000 +0200
>+++ ./poppler/GfxFont.cc	2006-06-12 14:30:21.000000000 +0200
>@@ -147,6 +147,7 @@ GfxFont::GfxFont(char *tagA, Ref idA, Go
>   family = NULL;
>   stretch = StretchNotDefined;
>   weight = WeightNotDefined;
>+  refCnt = 1;
> }
> 
> GfxFont::~GfxFont() {
>@@ -166,6 +167,15 @@ GfxFont::~GfxFont() {
>   }
> }
> 
>+void GfxFont::incRefCnt() {
>+  refCnt++;
>+}
>+
>+void GfxFont::decRefCnt() {
>+  if (--refCnt == 0)
>+    delete this;
>+}
>+
> void GfxFont::readFontDescriptor(XRef *xref, Dict *fontDict) {
>   Object obj1, obj2, obj3, obj4;
>   double t;
>@@ -459,6 +469,7 @@ Gfx8BitFont::Gfx8BitFont(XRef *xref, cha
>   Object obj1, obj2, obj3;
>   int n, i, a, b, m;
> 
>+  refCnt = 1;
>   type = typeA;
>   ctu = NULL;
> 
>@@ -1086,6 +1097,7 @@ GfxCIDFont::GfxCIDFont(XRef *xref, char 
>   int c1, c2;
>   int excepsSize, i, j, k;
> 
>+  refCnt = 1;
>   ascent = 0.95;
>   descent = -0.35;
>   fontBBox[0] = fontBBox[1] = fontBBox[2] = fontBBox[3] = 0;
>@@ -1530,7 +1542,7 @@ GfxFontDict::~GfxFontDict() {
> 
>   for (i = 0; i < numFonts; ++i) {
>     if (fonts[i]) {
>-      delete fonts[i];
>+      fonts[i]->decRefCnt();
>     }
>   }
>   gfree(fonts);
>diff -rup ../poppler-0.4.5/poppler/GfxFont.h ./poppler/GfxFont.h
>--- ../poppler-0.4.5/poppler/GfxFont.h	2005-07-06 19:12:36.000000000 +0200
>+++ ./poppler/GfxFont.h	2006-06-12 14:30:21.000000000 +0200
>@@ -115,6 +115,9 @@ public:
> 
>   GBool isOk() { return ok; }
> 
>+  void incRefCnt();
>+  void decRefCnt();
>+
>   // Get font tag.
>   GooString *getTag() { return tag; }
> 
>@@ -215,6 +218,7 @@ protected:
>   double missingWidth;		// "default" width
>   double ascent;		// max height above baseline
>   double descent;		// max depth below baseline
>+  int refCnt;
>   GBool ok;
> };
> 
>diff -rup ../poppler-0.4.5/poppler/GfxState.cc ./poppler/GfxState.cc
>--- ../poppler-0.4.5/poppler/GfxState.cc	2005-08-06 00:30:02.000000000 +0200
>+++ ./poppler/GfxState.cc	2006-06-12 14:31:16.000000000 +0200
>@@ -21,6 +21,7 @@
> #include "Array.h"
> #include "Page.h"
> #include "GfxState.h"
>+#include "GfxFont.h"
> 
> //------------------------------------------------------------------------
> 
>@@ -2676,6 +2677,9 @@ GfxState::~GfxState() {
>   if (saved) {
>     delete saved;
>   }
>+  if (font) {
>+    font->decRefCnt();
>+  }
> }
> 
> // Used for copy();
>@@ -2697,6 +2701,9 @@ GfxState::GfxState(GfxState *state) {
>     lineDash = (double *)gmalloc(lineDashLength * sizeof(double));
>     memcpy(lineDash, state->lineDash, lineDashLength * sizeof(double));
>   }
>+  if (font)
>+    font->incRefCnt();
>+
>   saved = NULL;
> }
> 
>diff -rup ../poppler-0.4.5/poppler/TextOutputDev.cc ./poppler/TextOutputDev.cc
>--- ../poppler-0.4.5/poppler/TextOutputDev.cc	2005-12-12 23:26:24.000000000 +0100
>+++ ./poppler/TextOutputDev.cc	2006-06-12 15:02:04.000000000 +0200
>@@ -118,6 +118,8 @@
> 
> TextFontInfo::TextFontInfo(GfxState *state) {
>   gfxFont = state->getFont();
>+  if (gfxFont)
>+    gfxFont->incRefCnt ();
> #if TEXTOUT_WORD_LIST
>   fontName = (gfxFont && gfxFont->getOrigName())
>                  ? gfxFont->getOrigName()->copy()
>@@ -126,6 +128,8 @@ TextFontInfo::TextFontInfo(GfxState *sta
> }
> 
> TextFontInfo::~TextFontInfo() {
>+  if (gfxFont)
>+    gfxFont->decRefCnt ();
> #if TEXTOUT_WORD_LIST
>   if (fontName) {
>     delete fontName;
>@@ -3298,6 +3302,7 @@ void TextSelectionPainter::visitWord (Te
> 
>   state->setFillColor(glyph_color);
>   out->updateFillColor(state);
>+  word->font->gfxFont->incRefCnt();
>   state->setFont(word->font->gfxFont, word->fontSize);
>   out->updateFont(state);
>

Actions: View | Diff

Attachments on bug 152780: 70360 | 88664 | 88757