RE: [xslt] small patch for xsltFreeStylePreComps()


> -----Original Message-----
> From: xslt-bounces gnome org [mailto:xslt-bounces gnome org] 
> On Behalf Of Aleksey Sanin
> Thanks!
> Aleksey
> Buchcik, Kasimier wrote:
> > Hi Aleksey,
> > 
> > Missed that Daniel was deferring to me, when I read your mail
> > the first time.
> > 
> > The only thing that we need additionally to take care of, is
> > the order of freeing: in xsltFreeStylesheet() the documents
> > of 1 stylesheet-level are freed before the compiled information
> > is freed. So we need to reverse that order.
> > 
> > I've added the patch to my local changed and hope to commit
> > this today with all the refactored stuff (still IFDEFed out


Problem: Any clue how to also clear the "xsltExtMarker" (in preproc.h)
from the "psvi" field? It is set on extension elements which were not
compiled during compile-time.
Additionally, I introduced an "xsltXSLTTextMarker" to be set; and
I would like to add an "xsltLiteralResultElemMarker" as well.

Wouldn't be an explicit call to a cleanup function for stylesheet-docs
be more robust here? Should be fast enough to traverse the tree
non-recursively, descending only into element-nodes and clearing
the "psvi" fields. Even a built-in call to such a cleanup function
would be OK with me, since really big stylehssets are mostly split into
includes and imports, which aren't accessible to the caller (they are
loaded internally), thus we only need to cleanup the the main
stylesheet's doc. Would that be an option for you?


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]