[poppler] [PATCH 0/5] Various fixes relating to bug #24575

[Albert Astals Cid]

A Divendres, 22 de gener de 2010, David A Benjamin va escriure:
> Oops. Sorry guys; forgot to set the subject.

Sorry to be a pain, but could you try not to use GooVector, that class has a 
"unknown" origin and we don't use anywhere other than pdftohtml and i'd prefer 
to keep it that way.

Albert

> 
> On Fri, 22 Jan 2010, David Benjamin wrote:
> > So, bug #24575 had a patch sent in that partially fixed things which was
> > then incorrectly rewritten and refixed again, but that one has a suble
> > bug left, and none of these really correctly fixed the stack handling,
> > and at the end of the day, the PDF still doesn't render.
> >
> > This patchset should fix all these issues. It falls prey to the same
> > problem of changing xpdf code as the original patch did, but I think the
> > stack handling is worth it; this is really the right place to put the
> > code, and the merge should not be too complicated in the unlikely event
> > that xpdf ever releases again. If people really think it's a problem, I
> > can personally offer to fix those merges.
> >
> > Hand-crafted test-cases for the last commit may be found in my Public:
> >
> > http://web.mit.edu/davidben/Public/poppler/
> >
> > David Benjamin (5):
> >  Avoid leaving pointers to free'd memory
> >  Do not use objects just after deleting them
> >  Maintain the height of the stack in Gfx
> >  Add a stack of stateGuards to Gfx
> >  Allow commands to abort the current stream
> >
> > poppler/CairoOutputDev.cc |    5 ++---
> > poppler/Gfx.cc            |   43
> > +++++++++++++++++++++++++++++++++++++++++++ poppler/Gfx.h             |  
> > 12 ++++++++++++
> > poppler/GfxFont.cc        |    6 ++++--
> > 4 files changed, 61 insertions(+), 5 deletions(-)
> 
> _______________________________________________
> poppler mailing list
> poppler at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/poppler
>