You are viewing a plain text version of this content. The canonical link for it is here.
Posted to ivy-dev@incubator.apache.org by Ja...@rzf.fin-nrw.de on 2007/10/29 12:10:50 UTC

AW: svn commit: r589579 - /incubator/ivy/core/trunk/src/java/org/apache/ivy/Main.java

>-    private static List getExtraClasspathFileList(CommandLine line) {
>+    /**
>+     * Parses the <code>cp</code> option from the command line, and
returns a list of {@link File}.
>+     * <p>
>+     * All the files contained in the returned List exist, non
existing files are simply skipped
>+     * with a warning.

need a </p> here ...


>+     * 
>+     * @param line
>+     *            the command line in which the cp option shold be
parsed
>+     * @return a List of files to include as extra classpath entries,
or <code>null</code> if no
>+     *         cp option was provided.
>+     */

>+    private static List/*<File>*/
getExtraClasspathFileList(CommandLine line) {
>-            fileList = new ArrayList();
>+            fileList = new ArrayList/*<File>*/();

nice Java5-comment ;)


Jan

Re: svn commit: r589579 - /incubator/ivy/core/trunk/src/java/org/apache/ivy/Main.java

Posted by Xavier Hanin <xa...@gmail.com>.
On 10/29/07, Jan.Materne@rzf.fin-nrw.de <Ja...@rzf.fin-nrw.de> wrote:
>
> >-    private static List getExtraClasspathFileList(CommandLine line) {
> >+    /**
> >+     * Parses the <code>cp</code> option from the command line, and
> returns a list of {@link File}.
> >+     * <p>
> >+     * All the files contained in the returned List exist, non
> existing files are simply skipped
> >+     * with a warning.
>
> need a </p> here ...


fixed

Xavier

>+     *
> >+     * @param line
> >+     *            the command line in which the cp option shold be
> parsed
> >+     * @return a List of files to include as extra classpath entries,
> or <code>null</code> if no
> >+     *         cp option was provided.
> >+     */
>
> >+    private static List/*<File>*/
> getExtraClasspathFileList(CommandLine line) {
> >-            fileList = new ArrayList();
> >+            fileList = new ArrayList/*<File>*/();
>
> nice Java5-comment ;)
>
>
> Jan
>



-- 
Xavier Hanin - Independent Java Consultant
http://xhab.blogspot.com/
http://ant.apache.org/ivy/
http://www.xoocode.org/