Re: Comment on TW's changeset #371

[percious]

Agree,

I am not sure the right way to implement this however. Admittedly, I
borrowed the idea from pylons, so your suggestion may be valuable
there as well:

http://bitbucket.org/bbangert/pylons/src/tip/pylons/controllers/core.py

cheers.
-chris

On May 18, 12:28 am, Alberto Valverde <albe...@toscat.net> wrote:
> Hi,
>
> http://toscawidgets.org/trac/tw/changeset/371
>
> Perhaps ChachedInspect could store weak-references to the keys in the
> self.funcions, self.methods and self.argspec dicts in order not to cause
> memory leaks? The problem is that params passed to widgets when
> displaying them tend to fall out of scope pretty quickly since they're
> usually created inside the views/controllers on every request. The
> current implementation prevents them from being gc'ed.
>
> Cheers,
> Alberto

[Ben Bangert]

On May 28, 2009, at 2:59 PM, percious wrote:

> Agree,
>
> I am not sure the right way to implement this however. Admittedly, I
> borrowed the idea from pylons, so your suggestion may be valuable
> there as well:
>
> http://bitbucket.org/bbangert/pylons/src/tip/pylons/controllers/
> core.py

Is this because functions are being dynamically added/removed from the
controller, and the cached inspect is holding onto them? Any
elaboration on what exactly is occurring?

Cheers,
Ben