search

gintool / gin

GI in No Time - a Simple Microframework for Genetic Improvement
MIT License
43 stars 21 forks source link

FullyQualifiedNames.getClassName() confused by inner classes #65

Closed sandybrownlee closed 4 years ago

sandybrownlee commented 4 years ago

It seems that if inner classes appear before the main class declaration, this method can pick those up instead, with can throw off detection of target methods.

e.g. in Disruptor, trying to target com.lmax.disruptor.Sequence.compareAndSet(long,long) gets broken because Sequence.java contains classes LhsPadding,Value, and RhsPadding before Sequence.

sandybrownlee commented 4 years ago

woo hoo - seems to be fixed.