[jboss-jira] [JBoss JIRA] Commented: (JBAOP-568) Avoid extra info from being kept in the generated jp cache

Thu May 1 00:31:18 EDT 2008

    [ http://jira.jboss.com/jira/browse/JBAOP-568?page=comments#action_12411437 ] 

Flavia Rainone commented on JBAOP-568:
--------------------------------------

I'm thinking that the reference to the class loader should also be removed from this cache? After all, the class loader is always the same.
This class loader reference was added because of JBAOP-550. But, later on, we discovered that the behaviour seen in the Messaging tests (that took to the creation of JBAOP-550) were being caused by a bug in the class loader used by those tests.

> Avoid extra info from being kept in the generated jp cache
> ----------------------------------------------------------
>
>                 Key: JBAOP-568
>                 URL: http://jira.jboss.com/jira/browse/JBAOP-568
>             Project: JBoss AOP
>          Issue Type: Feature Request
>      Security Level: Public(Everyone can see) 
>    Affects Versions: 2.0.0.CR8
>            Reporter: Flavia Rainone
>         Assigned To: Flavia Rainone
>             Fix For: 2.0.0.CR9
>
>
> GeneratedClassInfo, object used to keep the generated JoinPoint cache in JoinPointGenerator instances, keeps references to AdviceMethodProperties.
> However, the single information that is really required from AdviceMethodProperties during generated JoinPoint instantiation are the around cflow field names and initialization expressions.
> If we stop from keeping references to AdviceMethodProperties inside this cache, we will make it available for garbage collection.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.jboss.com/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira