"Geir Magnusson Jr." wrote:
>
> From: John McNally [mailto:[EMAIL PROTECTED]]
>
> > This patch fixes a memory leak in texen, where the repeated use of the
> > context without caching templates leads to introspection cache growth.
>
> To be fair, it wasn't a memory leak, but rather a combination of factors
> that caused new introspection information
> to be repeatedly acquired and cached.
>
I was probably using memory leak in an imprecise manner as I think
java's garbage collection is supposed to eliminate the possibility. I
just meant it was possible to create a situation where what should
amount to a couple megabytes of valid data at any time could be
accumulated to create an out of memory error. To be really fair it
could be said that an application using texen should be sure to set
template caching on. A better fix would probably check the caching
property and provide the dummy proof Context wrapper if it is off, but
use the context directly if on, so that performance can be maximized.
But I was very happy to see that what seemed like a very hard problem
turned out to have a one line fix. :)
Thanks for your help,
john mcnally
