Steve Loughran wrote:

Peter Reilly wrote:

[EMAIL PROTECTED] wrote:

stevel      2005/03/22 09:20:06


+ public static final String WARN_JAR_AND_FILESET = "nested filesets will be ignored if the jar attribute has"
+ + " been specified.";
+ public static final String ERROR_BAD_MAP = "Cannot map source file to anything sensible: ";
+ public static final String ERROR_MAPPER_WITHOUT_DEST = "The destDir attribute is required if a mapper is set";
+ public static final String ERROR_NO_SOURCE = "jar must be set through jar attribute "
+ + "or nested filesets";
+ public static final String ERROR_NO_ALIAS = "alias attribute must be set";
+ public static final String ERROR_NO_STOREPASS = "storepass attribute must be set";


Is there a need for these to be public rather than private?
Peter


could make them package scoped. I want to grab them in the unit tests.

Ah, that makes sense. The reason I asked was because checkstyle whines about all the undocumented public strings.

Peter


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



Reply via email to