"timfox" wrote : "clebert.suconic(a)jboss.com" wrote : I would need to create a ClientMessageInternal interface for that, and I would need to probably change all the usages from ClientMessage to ClientMessageInternal. (Or else I would need to type-cast to ClientMessageInternal).
| |
| |
| Instead of spending 3 hours debating this, I just made the change myself.
|
| I timed myself, it took 2 minutes 30 second to refactor the code.
|
| It's in SVN now.
Well.. you don't need to convince yourself :-)
I wanted to double check to verify if you wouldn't disagree with that.
View the original post : http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4195713#4195713
Reply to the post : http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&p=4195713
"clebert.suconic(a)jboss.com" wrote : I would need to create a ClientMessageInternal interface for that, and I would need to probably change all the usages from ClientMessage to ClientMessageInternal. (Or else I would need to type-cast to ClientMessageInternal).
|
|
Instead of spending 3 hours debating this, I just made the change myself.
I timed myself, it took 2 minutes 30 second to refactor the code.
It's in SVN now.
View the original post : http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4195706#4195706
Reply to the post : http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&p=4195706
I don't think this is a problem for AOP since whenever we generate classes they will be using the package of one of the classes in the classloader used to generate the class, but I thought I'd point out my following finding.
I create a domain with 2 BaseClassLoaders. classLoaderA has package blah.a, and classLoaderB has package blah.b. Now if I use javassist to create a class called not.registered.package.SomeClasss in ClassLoaderA then
| //works since the class is registered in the loader's loaded classes
| classLoaderA.loadClass("not.registered.package.SomeClasss");
| //Does not work since not.registered.package is not in the
| //list of packages of any of the loaders registered in the domain
| classLoaderA.loadClass("not.registered.package.SomeClasss");
|
|
|
View the original post : http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4195684#4195684
Reply to the post : http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&p=4195684