>> >+    private static List 
>getExtraClasspathFileList(CommandLine line) {
>>
>> no javadoc?
>
>
>You're right, I should add some. The fact is that this commit comes
from a
>patch where there wasn't this method (its code was in the main method),
but
>the main method length then exceeded checkstyle threshold, thus I run
>refactor extract method just to make checkstyle happy, and forgot to
add the
>javadoc.

yeah, I can understand that ...
But we have a multiple-eyes-quality-control-mechanism here ;)



Jan

Reply via email to