Erik,

are there any requirements on 'fs'? What if it's null? What if the fileset
is empty?
--
dIon Gillard, Multitask Consulting
Work:      http://www.multitask.com.au
Developers: http://adslgateway.multitask.com.au/developers


                                                                                
                                              
                    "Erik Hatcher"                                              
                                              
                    <[EMAIL PROTECTED]       To:     "Ant Developers List" 
<[EMAIL PROTECTED]>                
                    tions.com>                      cc:                         
                                              
                                                    Subject:     Re: cvs 
commit:                                              
                    06/19/02 12:03 PM                
jakarta-ant/src/main/org/apache/tools/ant/taskdefs Ear.java Jar.java     
                    Please respond to "Ant           Zip.java                   
                                              
                    Developers List"                                            
                                              
                                                                                
                                              
                                                                                
                                              




> [EMAIL PROTECTED] wrote:
> >   +    /**
> >   +     * Adds zipfileset.
> >   +     *
> >   +     * @param fs
> >   +     */
> >        public void addArchives(ZipFileSet fs) {
> >            // We just set the prefix for this fileset, and pass it up.
> >            // Do we need to do this? LH
>
> Don't you want some description of fs?

Yes.  I'm trying to catch these.  IDEA is automatically adding the @params
for new Javadoc blocks. Its hard for me to bring myself to duplicate what
was just said above though.  I'll fix this one and others that I create.

Will <javadoc> complain if they are missing?  Or I'm assuming Checkstyle
would?

Thanks,
    Erik



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






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

Reply via email to