search

mktany2k / funcito

Automatically exported from code.google.com/p/funcito
0 stars 0 forks source link

strange inner class #21

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
Javassist.Imposterizer has an inner class called 
ClassWithSuperclassToWorkAroundCglibBug

Either this is unnecessary or poorly named? It is being used but I can't tell 
if it is a copy-paste error.

Original issue reported on code.google.com by codeto...@gmail.com on 25 Nov 2011 at 9:58

GoogleCodeExporter commented 9 years ago 
It was part of CGLib to Javassist conversion.  ClassImposterizer was a 
pre-exsiting class (from JMock?  EasyMock?, anyway, Mockito borrowed it and I 
got it from them), and I attempted to convert it for use with Javassist.  I am 
not sure whether this is needed for Javassist or not.  You are right that at 
the least the name is wrong.  I prolly need to write a test to see whether or 
not it is needed.

Original comment by kandpwel...@gmail.com on 29 Nov 2011 at 7:03

GoogleCodeExporter commented 9 years ago 
fixed in last checkin

Original comment by kandpwel...@gmail.com on 12 Dec 2011 at 6:58

GoogleCodeExporter commented 9 years ago 
changing status to fixed

Original comment by kandpwel...@gmail.com on 16 Dec 2011 at 5:55