Hi Dennis, Yes it was intentional because it is a redundant test (see line 1873). BTW, I would have to do 2 commits for this.
Cheers, Vincent 2007/11/3, Dennis Lundberg <[EMAIL PROTECTED]>: > [EMAIL PROTECTED] wrote: > > Author: vsiveton > > Date: Sat Nov 3 03:41:22 2007 > > New Revision: 591576 > > > > URL: http://svn.apache.org/viewvc?rev=591576&view=rev > > Log: > > o using Plexus to instantiate mojo parameters with default-value > > > > Modified: > > > > maven/plugins/trunk/maven-javadoc-plugin/src/main/java/org/apache/maven/plugin/javadoc/AbstractJavadocMojo.java > > > > Modified: > > maven/plugins/trunk/maven-javadoc-plugin/src/main/java/org/apache/maven/plugin/javadoc/AbstractJavadocMojo.java > > URL: > > http://svn.apache.org/viewvc/maven/plugins/trunk/maven-javadoc-plugin/src/main/java/org/apache/maven/plugin/javadoc/AbstractJavadocMojo.java?rev=591576&r1=591575&r2=591576&view=diff > > ============================================================================== > > --- > > maven/plugins/trunk/maven-javadoc-plugin/src/main/java/org/apache/maven/plugin/javadoc/AbstractJavadocMojo.java > > (original) > > +++ > > maven/plugins/trunk/maven-javadoc-plugin/src/main/java/org/apache/maven/plugin/javadoc/AbstractJavadocMojo.java > > Sat Nov 3 03:41:22 2007 > > @@ -281,7 +281,7 @@ > > * > > * @parameter expression="${breakiterator}" default-value="false" > > */ > > - private boolean breakiterator = false; > > + private boolean breakiterator; > > > > /** > > * Specifies the class file that starts the doclet used in generating > > the documentation. > > @@ -433,7 +433,7 @@ > > * > > * @parameter expression="${old}" default-value="false" > > */ > > - private boolean old = false; > > + private boolean old; > > > > /** > > * Specifies that javadoc should retrieve the text for the overview > > documentation from the "source" file > > @@ -463,7 +463,7 @@ > > * > > * @parameter expression="${show}" default-value="protected" > > */ > > - private String show = "protected"; > > + private String show; > > > > /** > > * Shuts off non-error and non-warning messages, leaving only the > > warnings and errors appear, making them > > @@ -478,7 +478,7 @@ > > * > > * @parameter expression="${quiet}" default-value="false" > > */ > > - private boolean quiet = false; > > + private boolean quiet; > > > > /** > > * Necessary to enable javadoc to handle assertions present in J2SE v > > 1.4 source code. > > @@ -520,7 +520,7 @@ > > * > > * @parameter expression="${verbose}" default-value="false" > > */ > > - private boolean verbose = false; > > + private boolean verbose; > > > > // > > ---------------------------------------------------------------------- > > // Standard Doclet Options > > @@ -533,7 +533,7 @@ > > * > > * @parameter expression="${author}" default-value="true" > > */ > > - private boolean author = true; > > + private boolean author; > > > > /** > > * Specifies the text to be placed at the bottom of each output > > file.<br/> > > @@ -554,7 +554,7 @@ > > * > > * @parameter expression="${charset}" default-value="ISO-8859-1" > > */ > > - private String charset = "ISO-8859-1"; > > + private String charset; > > > > /** > > * Specifies the destination directory where javadoc saves the > > generated HTML files. > > @@ -575,7 +575,7 @@ > > * > > * @parameter expression="${docfilessubdirs}" default-value="false" > > */ > > - private boolean docfilessubdirs = false; > > + private boolean docfilessubdirs; > > > > /** > > * Specifies the encoding of the generated HTML files. > > @@ -714,7 +714,7 @@ > > * > > * @parameter expression="${linksource}" default-value="false" > > */ > > - private boolean linksource = false; > > + private boolean linksource; > > > > /** > > * Suppress the entire comment body, including the main description > > and all tags, generating only declarations. > > @@ -725,7 +725,7 @@ > > * > > * @parameter expression="${nocomment}" default-value="false" > > */ > > - private boolean nocomment = false; > > + private boolean nocomment; > > > > /** > > * Prevents the generation of any deprecated API at all in the > > documentation. > > @@ -734,7 +734,7 @@ > > * > > * @parameter expression="${nodeprecated}" default-value="false" > > */ > > - private boolean nodeprecated = false; > > + private boolean nodeprecated; > > > > /** > > * Prevents the generation of the file containing the list of > > deprecated APIs (deprecated-list.html) and the > > @@ -745,7 +745,7 @@ > > * > > * @parameter expression="${nodeprecatedlist}" default-value="false" > > */ > > - private boolean nodeprecatedlist = false; > > + private boolean nodeprecatedlist; > > > > /** > > * Omits the HELP link in the navigation bars at the top and bottom of > > each page of output. > > @@ -756,7 +756,7 @@ > > * > > * @parameter expression="${nohelp}" default-value="false" > > */ > > - private boolean nohelp = false; > > + private boolean nohelp; > > > > /** > > * Omits the index from the generated docs. > > @@ -767,7 +767,7 @@ > > * > > * @parameter expression="${noindex}" default-value="false" > > */ > > - private boolean noindex = false; > > + private boolean noindex; > > > > /** > > * Omits the navigation bar from the generated docs. > > @@ -776,7 +776,7 @@ > > * > > * @parameter expression="${nonavbar}" default-value="false" > > */ > > - private boolean nonavbar = false; > > + private boolean nonavbar; > > > > /** > > * Omits the entire overview page from the generated docs. > > @@ -788,7 +788,7 @@ > > * @since 2.4 > > * @parameter expression="${nooverview}" default-value="false" > > */ > > - private boolean nooverview = false; > > + private boolean nooverview; > > > > /** > > * Omits qualifying package name from ahead of class names in output. > > @@ -808,7 +808,7 @@ > > * > > * @parameter expression="${nosince}" default-value="false" > > */ > > - private boolean nosince = false; > > + private boolean nosince; > > > > /** > > * Suppresses the timestamp, which is hidden in an HTML comment in the > > generated HTML near the top of each page. > > @@ -830,7 +830,7 @@ > > * > > * @parameter expression="${notree}" default-value="false" > > */ > > - private boolean notree = false; > > + private boolean notree; > > > > /** > > * Specify the text for upper left frame. > > @@ -850,7 +850,7 @@ > > * > > * @parameter expression="${serialwarn}" default-value="false" > > */ > > - private boolean serialwarn = false; > > + private boolean serialwarn; > > > > /** > > * Specify the number of spaces each tab takes up in the source. If no > > tab is used in source, the default > > @@ -879,7 +879,7 @@ > > * > > * @parameter expression="${splitindex}" default-value="false" > > */ > > - private boolean splitindex = false; > > + private boolean splitindex; > > > > /** > > * Specifies whether the stylesheet to be used is the maven javadoc > > stylesheet or java's default stylesheet > > @@ -1010,7 +1010,7 @@ > > * > > * @parameter expression="${use}" default-value="true" > > */ > > - private boolean use = true; > > + private boolean use; > > > > /** > > * Includes the version text in the generated docs. > > @@ -1019,7 +1019,7 @@ > > * > > * @parameter expression="${version}" default-value="true" > > */ > > - private boolean version = true; > > + private boolean version; > > > > /** > > * Specifies the title to be placed in the HTML title tag. > > @@ -1876,8 +1876,7 @@ > > } > > else > > { > > - if ( ( project.getOrganization() != null ) > > - && ( StringUtils.isNotEmpty( > > project.getOrganization().getName() ) ) ) > > + if ( StringUtils.isNotEmpty( > > project.getOrganization().getName() ) ) > > { > > if ( StringUtils.isNotEmpty( > > project.getOrganization().getUrl() ) ) > > { > > This will open up for a possible NPE if project.getOrganization() is > null... Was that intentional? > > -- > Dennis Lundberg > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > > --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]