Jason, Why you don't use constants?
Emmanuel ----- Original Message ----- From: <[EMAIL PROTECTED]> To: <[EMAIL PROTECTED]> Sent: Monday, March 22, 2004 3:34 PM Subject: cvs commit: maven-components/maven-core/src/main/java/org/apache/maven/plugin/descriptor PluginDescriptor.java > jvanzyl 2004/03/22 06:34:12 > > Modified: maven-core/src/main/java/org/apache/maven/plugin/descriptor > PluginDescriptor.java > Log: > > > Revision Changes Path > 1.4 +2 -9 maven-components/maven-core/src/main/java/org/apache/maven/plugin/descriptor /PluginDescriptor.java > > Index: PluginDescriptor.java > =================================================================== > RCS file: /home/cvs/maven-components/maven-core/src/main/java/org/apache/maven/plugin/ descriptor/PluginDescriptor.java,v > retrieving revision 1.3 > retrieving revision 1.4 > diff -u -r1.3 -r1.4 > --- PluginDescriptor.java 21 Mar 2004 13:33:58 -0000 1.3 > +++ PluginDescriptor.java 22 Mar 2004 14:34:12 -0000 1.4 > @@ -21,13 +21,6 @@ > > import java.util.List; > > -/** > - * > - * > - * @author <a href="mailto:[EMAIL PROTECTED]">Jason van Zyl</a> > - * > - * @version $Id$ > - */ > public class PluginDescriptor > extends ComponentDescriptor > { > @@ -35,7 +28,7 @@ > > private List goals; > > - private String mode = Plugin.FIELD; > + private String mode = "field"; > > public String getRole() > { > @@ -44,7 +37,7 @@ > > public String getInstantiationStrategy() > { > - if ( mode.equals( Plugin.FIELD ) || mode.equals( Plugin.SETTER ) ) > + if ( mode.equals( "field" ) || mode.equals( "setter" ) ) > { > return "per-lookup"; > } > > > > > --------------------------------------------------------------------- > 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]