You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ant.apache.org by pe...@apache.org on 2004/12/13 18:56:15 UTC
cvs commit: ant/src/main/org/apache/tools/ant/taskdefs/rmic DefaultRmicAdapter.java
peterreilly 2004/12/13 09:56:15
Modified: src/main/org/apache/tools/ant/taskdefs/rmic
DefaultRmicAdapter.java
Log:
checkstyle changes
Obtained from: Kevin Jackson
Revision Changes Path
1.35 +41 -9 ant/src/main/org/apache/tools/ant/taskdefs/rmic/DefaultRmicAdapter.java
Index: DefaultRmicAdapter.java
===================================================================
RCS file: /home/cvs/ant/src/main/org/apache/tools/ant/taskdefs/rmic/DefaultRmicAdapter.java,v
retrieving revision 1.34
retrieving revision 1.35
diff -u -r1.34 -r1.35
--- DefaultRmicAdapter.java 8 Oct 2004 11:09:46 -0000 1.34
+++ DefaultRmicAdapter.java 13 Dec 2004 17:56:15 -0000 1.35
@@ -37,31 +37,57 @@
private Rmic attributes;
private FileNameMapper mapper;
- private static final Random rand = new Random();
+ private static final Random RAND = new Random();
+ /** suffix denoting a stub file */
public static final String RMI_STUB_SUFFIX = "_Stub";
+ /** suffix denoting a skel file */
public static final String RMI_SKEL_SUFFIX = "_Skel";
+ /** suffix denoting a tie file */
public static final String RMI_TIE_SUFFIX = "_Tie";
+ /**
+ * Default constructor
+ */
public DefaultRmicAdapter() {
}
- public void setRmic(Rmic attributes) {
+ /**
+ * Sets Rmic attributes
+ * @param attributes the rmic attributes
+ */
+ public void setRmic(final Rmic attributes) {
this.attributes = attributes;
mapper = new RmicFileNameMapper();
}
+ /**
+ * Get the Rmic attributes
+ * @return the attributes as a Rmic taskdef
+ */
public Rmic getRmic() {
return attributes;
}
+ /**
+ * Gets the stub class suffix
+ * @return the stub suffix "_Stub"
+ */
protected String getStubClassSuffix() {
return RMI_STUB_SUFFIX;
}
+ /**
+ * Gets the skeleton class suffix
+ * @return the skeleton suffix "_Skel"
+ */
protected String getSkelClassSuffix() {
return RMI_SKEL_SUFFIX;
}
+ /**
+ * Gets the tie class suffix
+ * @return the tie suffix "_Tie"
+ */
protected String getTieClassSuffix() {
return RMI_TIE_SUFFIX;
}
@@ -81,13 +107,15 @@
* interfaces and _*_getStubClassSuffix for non-interfaces (and
* determine the interface and create _*_Stub from that).</li>
* </ul>
+ * @return a <code>FileNameMapper</code>
*/
public FileNameMapper getMapper() {
return mapper;
}
/**
- * The CLASSPATH this rmic process will use.
+ * Gets the CLASSPATH this rmic process will use.
+ * @return the classpath
*/
public Path getClasspath() {
return getCompileClasspath();
@@ -95,6 +123,7 @@
/**
* Builds the compilation classpath.
+ * @return the classpath
*/
protected Path getCompileClasspath() {
Path classpath = new Path(attributes.getProject());
@@ -122,17 +151,18 @@
}
/**
- * setup rmic argument for rmic.
+ * Setup rmic argument for rmic.
+ * @return the command line
*/
protected Commandline setupRmicCommand() {
return setupRmicCommand(null);
}
/**
- * setup rmic argument for rmic.
- *
+ * Setup rmic argument for rmic.
* @param options additional parameters needed by a specific
* implementation.
+ * @return the command line
*/
protected Commandline setupRmicCommand(String[] options) {
Commandline cmd = new Commandline();
@@ -204,6 +234,7 @@
/**
* Logs the compilation parameters, adds the files to compile and logs the
* "niceSourceList"
+ * @param cmd the commandline args
*/
protected void logAndAddFilesToCompile(Commandline cmd) {
Vector compileList = attributes.getCompileList();
@@ -212,12 +243,13 @@
Project.MSG_VERBOSE);
StringBuffer niceSourceList = new StringBuffer("File");
- if (compileList.size() != 1) {
+ int cListSize = compileList.size();
+ if (cListSize != 1) {
niceSourceList.append("s");
}
niceSourceList.append(" to be compiled:");
- for (int i = 0; i < compileList.size(); i++) {
+ for (int i = 0; i < cListSize; i++) {
String arg = (String) compileList.elementAt(i);
cmd.createArgument().setValue(arg);
niceSourceList.append(" " + arg);
@@ -284,7 +316,7 @@
* This is supposed to make Ant always recompile the
* class, as a file of that name should not exist.
*/
- String[] target = new String[] {name + ".tmp." + rand.nextLong()};
+ String[] target = new String[] {name + ".tmp." + RAND.nextLong()};
if (!attributes.getIiop() && !attributes.getIdl()) {
// JRMP with simple naming convention
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
For additional commands, e-mail: dev-help@ant.apache.org