Community
Participate
Working Groups
At the moment the way the constructor ipes are constructed is different to that of methods. For example, for a method which has signature public void myMethod(String s, int i) the corresponding ipe is firstly constructed with name "myMethod" and then the arguments are added. Adding the arguments results in an ipe with name "myMethod(java.lang.String,int)". With a constructor that has signature public MyClass(String s, int i) the corresponding ipe is constructed with name "MyClass(String,int)" and no arguments are associated with it. Up until now there hasn't been a use case to change this. However, due to enhancement bug 141730 (updating handles not to rely on source locations) it is necessary for the constructor ipes to have arguments associated with them. Also, if the ipes of methods and constructors had similar properties then the processing for creating the new handles can be the same.
Created attachment 42704 [details] proposed fix This zip file contains four patches: - pr143930-ajdoc-patch.txt: apply to the ajdoc project - pr143930-ajdt-core-patch.txt: apply to the org.aspectj.ajdt.core project - pr143930-tests-patch.txt: apply to the tests project - pr143930-weaver-patch.txt: apply to the weaver project Along with the fix for the current bug, this contains the modified fix to bug 143924 mentioned in that bug report along with the test also supplied in that bug report.
patches committed.