DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUGĀ·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=36196>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED ANDĀ·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=36196





------- Additional Comments From [EMAIL PROTECTED]  2005-08-16 11:58 -------
Another issue with the EclipseJavaCompiler around line 130:

for (int i = 0; i < packageName.length; i++) {
    if (i != 0) {
        result.append('.');
    }
    result.append(packageName[i]);
}
result.append('.');
result.append(typeName);

IMO this code could be optimized to:

for (int i = 0; i < packageName.length; i++) {
    result.append(packageName[i]);
    result.append('.');
}
result.append(typeName);

But this would change the result for an empty packageName array. While code 1
returns ("." + typeName), code 2 would return just (typeName). I don't know what
the contract of this method is, but having a class name starting with a dot does
not sound that logical. So the optimized code might even fix this issue.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to