Re: cvs commit: ant/src/main/org/apache/tools/ant/taskdefs Checksum.java

2005-01-06 Thread Peter Reilly
Most of these are setX(x) where x is already defined in the javadoc. So there is no point in repeating the definition. (see the javadoc for most of the jdk). The buildexception statement is a bit meaningless, but it does capture what will happen. Peter Martijn Kruithof wrote: I understand the impor

Re: cvs commit: ant/src/main/org/apache/tools/ant/taskdefs Checksum.java

2005-01-06 Thread Martijn Kruithof
I understand the importance of improving the style, but shouldn't we prefer no comments (warnings stay to indicate they are missing) over this kind of comments: Kind Regards, Martijn + * @param file a File value + * @param algorithm a String value + * @param provider a String value

cvs commit: ant/src/main/org/apache/tools/ant/taskdefs Checksum.java

2005-01-06 Thread peterreilly
peterreilly2005/01/06 03:15:37 Modified:src/main/org/apache/tools/ant/taskdefs Checksum.java Log: checkstyle Revision ChangesPath 1.43 +37 -6 ant/src/main/org/apache/tools/ant/taskdefs/Checksum.java Index: Checksum.java

cvs commit: ant/src/main/org/apache/tools/ant/taskdefs Checksum.java ImportTask.java Redirector.java Recorder.java Untar.java

2005-01-06 Thread peterreilly
peterreilly2005/01/06 03:11:04 Modified:src/main/org/apache/tools/ant/taskdefs Checksum.java ImportTask.java Redirector.java Recorder.java Untar.java Log: remove imports Obtained from: Kevin Jackson Revision ChangesPath 1.

cvs commit: ant/src/main/org/apache/tools/ant/taskdefs Checksum.java

2003-06-24 Thread bodewig
bodewig 2003/06/24 08:28:23 Modified:docs/manual/CoreTasks checksum.html src/main/org/apache/tools/ant/taskdefs Checksum.java Log: 2003 and some 'since' information Revision ChangesPath 1.8 +2 -0 ant/docs/manual/CoreTasks/checksum.html Index