[poppler] Making OCG suppression be recursive

Albert Astals Cid aacid at kde.org
Wed May 6 10:48:48 PDT 2009


A Dimarts, 5 de maig de 2009, Carlos Garcia Campos va escriure:
> El lun, 04-05-2009 a las 00:22 +0200, Albert Astals Cid escribió:
> > Hi the attached patch adds that if we do a Begin Marked Content that is
> > suppressed and then we do a Begin Marked Content that is not suppressed
> > without closing the first one, the operations are still not drawn.
> >
> > This fixes bug 16093, i think it's the correct way but as Carlos and Brad
> > have worked more on Optional Content i'd like some more input if possible
> > (i know i'm asking for much :-))
>
> looks good to me.

Good, anyway i'm passing it through the regression suite i have here to see if 
it creates any problem or fixes any other bug.

Albert

>
> > Albert
> >
> >
> > diff --git a/poppler/Gfx.cc b/poppler/Gfx.cc
> > index 0eef5d3..b63d424 100644
> > --- a/poppler/Gfx.cc
> > +++ b/poppler/Gfx.cc
> > @@ -4216,7 +4216,13 @@ void Gfx::pushMarkedContent() {
> >  }
> >
> >  GBool Gfx::contentIsHidden() {
> > -  return mcStack && mcStack->ocSuppressed;
> > +  MarkedContentStack *mc = mcStack;
> > +  bool hidden = mc && mc->ocSuppressed;
> > +  while (!hidden && mc && mc->next) {
> > +    mc = mc->next;
> > +    hidden = mc->ocSuppressed;
> > +  }
> > +  return hidden;
> >  }
> >
> >  void Gfx::opBeginMarkedContent(Object args[], int numArgs) {
> >
> > _______________________________________________
> > poppler mailing list
> > poppler at lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/poppler




More information about the poppler mailing list