You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@xalan.apache.org by sh...@e-z.net on 2011/10/10 20:58:57 UTC

XALANC - XSLTEngineImpl Memory Leak?

Is there a memory problem here? -- analysis needed!

XSLT/XSLTEngineImpl.c

Constructor  XSLTEngineImpl::XSLTEngineImpl(...) initializes

   m_topLevelParams(theManager)
   m_stylesheetParams(theManager)

But the reset XSLTEngineImpl::reset() only clears

   m_topLevelParams.clear();

Should we also add the following to the reset()?

   m_stylesheetParams.clear();

-----------
Steven J.Hathaway


---------------------------------------------------------------------
To unsubscribe, e-mail: xalan-dev-unsubscribe@xml.apache.org
For additional commands, e-mail: xalan-dev-help@xml.apache.org


Re: XALANC - XSLTEngineImpl Memory Leak?

Posted by Steve Hathaway <sh...@e-z.net>.
There is no memory leak.

The m_stylesheetParams are intended to be sticky.

The user application should perform the m_stylesheetParams.clear()
before assigning new top-level parameters.

The reset() function is intended for reuse of an existing
XSLTEngineImpl instance for a new transformation and
preserves m_stylesheetParams so the structure does not
have to be reloaded between transformations using the
same class XSLTEngineImpl instance.

  - Steve

On 10/10/2011 11:58 AM, shathawa@e-z.net wrote:
> Is there a memory problem here? -- analysis needed!
>
> XSLT/XSLTEngineImpl.c
>
> Constructor  XSLTEngineImpl::XSLTEngineImpl(...) initializes
>
>     m_topLevelParams(theManager)
>     m_stylesheetParams(theManager)
>
> But the reset XSLTEngineImpl::reset() only clears
>
>     m_topLevelParams.clear();
>
> Should we also add the following to the reset()?
>
>     m_stylesheetParams.clear();
>
> -----------
> Steven J.Hathaway
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: xalan-dev-unsubscribe@xml.apache.org
> For additional commands, e-mail: xalan-dev-help@xml.apache.org
>


---------------------------------------------------------------------
To unsubscribe, e-mail: xalan-dev-unsubscribe@xml.apache.org
For additional commands, e-mail: xalan-dev-help@xml.apache.org