There are more locations which could benefit from not using a toCharArray on a String, but rather use the charAt method from the String itself. This to prevent an additional copy of the char[] being created.
Comment From: mdeinum
Additional note, while inspecting the AspectJAdviceParameterNameDiscoverer
I noticed that it might be beneficial to extract the isVariableName
method from the AbstractAspectJAdvice
to a helper/utility class and use that from both classes. They share the same logic to determine if something is a variable name.