You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2008/04/10 23:52:02 UTC
svn commit: r646980 - in /tomcat/trunk/java/org/apache/catalina/ssi:
ExpressionParseTree.java SSIExternalResolver.java SSIMediator.java
SSIPrintenv.java SSIProcessor.java SSIServletExternalResolver.java
Author: markt
Date: Thu Apr 10 14:51:57 2008
New Revision: 646980
URL: http://svn.apache.org/viewvc?rev=646980&view=rev
Log:
Java 5 clean up. No functional change.
Modified:
tomcat/trunk/java/org/apache/catalina/ssi/ExpressionParseTree.java
tomcat/trunk/java/org/apache/catalina/ssi/SSIExternalResolver.java
tomcat/trunk/java/org/apache/catalina/ssi/SSIMediator.java
tomcat/trunk/java/org/apache/catalina/ssi/SSIPrintenv.java
tomcat/trunk/java/org/apache/catalina/ssi/SSIProcessor.java
tomcat/trunk/java/org/apache/catalina/ssi/SSIServletExternalResolver.java
Modified: tomcat/trunk/java/org/apache/catalina/ssi/ExpressionParseTree.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/ssi/ExpressionParseTree.java?rev=646980&r1=646979&r2=646980&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/ssi/ExpressionParseTree.java (original)
+++ tomcat/trunk/java/org/apache/catalina/ssi/ExpressionParseTree.java Thu Apr 10 14:51:57 2008
@@ -31,12 +31,12 @@
* Contains the current set of completed nodes. This is a workspace for the
* parser.
*/
- private LinkedList nodeStack = new LinkedList();
+ private LinkedList<Node> nodeStack = new LinkedList<Node>();
/**
* Contains operator nodes that don't yet have values. This is a workspace
* for the parser.
*/
- private LinkedList oppStack = new LinkedList();
+ private LinkedList<OppNode> oppStack = new LinkedList<OppNode>();
/**
* The root node after the expression has been parsed.
*/
@@ -78,7 +78,7 @@
}
while (true) {
if (oppStack.size() == 0) break;
- OppNode top = (OppNode)oppStack.get(0);
+ OppNode top = oppStack.get(0);
// If the top is a spacer then don't pop
// anything
if (top == null) break;
@@ -103,7 +103,7 @@
*/
private void resolveGroup() {
OppNode top = null;
- while ((top = (OppNode)oppStack.remove(0)) != null) {
+ while ((top = oppStack.remove(0)) != null) {
// Let it fill its branches
top.popValues(nodeStack);
// Stick it on the resolved node stack
@@ -195,7 +195,7 @@
if (oppStack.size() != 0) {
throw new ParseException("Unused opp nodes exist.", et.getIndex());
}
- root = (Node)nodeStack.get(0);
+ root = nodeStack.get(0);
}
/**
@@ -272,9 +272,9 @@
* Lets the node pop its own branch nodes off the front of the
* specified list. The default pulls two.
*/
- public void popValues(List values) {
- right = (Node)values.remove(0);
- left = (Node)values.remove(0);
+ public void popValues(List<Node> values) {
+ right = values.remove(0);
+ left = values.remove(0);
}
}
private final class NotNode extends OppNode {
@@ -291,8 +291,8 @@
/**
* Overridden to pop only one value.
*/
- public void popValues(List values) {
- left = (Node)values.remove(0);
+ public void popValues(List<Node> values) {
+ left = values.remove(0);
}
Modified: tomcat/trunk/java/org/apache/catalina/ssi/SSIExternalResolver.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/ssi/SSIExternalResolver.java?rev=646980&r1=646979&r2=646980&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/ssi/SSIExternalResolver.java (original)
+++ tomcat/trunk/java/org/apache/catalina/ssi/SSIExternalResolver.java Thu Apr 10 14:51:57 2008
@@ -34,7 +34,7 @@
* @param variableNames
* the collection to add to
*/
- public void addVariableNames(Collection variableNames);
+ public void addVariableNames(Collection<String> variableNames);
public String getVariableValue(String name);
Modified: tomcat/trunk/java/org/apache/catalina/ssi/SSIMediator.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/ssi/SSIMediator.java?rev=646980&r1=646979&r2=646980&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/ssi/SSIMediator.java (original)
+++ tomcat/trunk/java/org/apache/catalina/ssi/SSIMediator.java Thu Apr 10 14:51:57 2008
@@ -125,8 +125,8 @@
}
- public Collection getVariableNames() {
- Set variableNames = new HashSet();
+ public Collection<String> getVariableNames() {
+ Set<String> variableNames = new HashSet<String>();
//These built-in variables are supplied by the mediator ( if not
// over-written by
// the user ) and always exist
@@ -135,9 +135,9 @@
variableNames.add("LAST_MODIFIED");
ssiExternalResolver.addVariableNames(variableNames);
//Remove any variables that are reserved by this class
- Iterator iter = variableNames.iterator();
+ Iterator<String> iter = variableNames.iterator();
while (iter.hasNext()) {
- String name = (String)iter.next();
+ String name = iter.next();
if (isNameReserved(name)) {
iter.remove();
}
Modified: tomcat/trunk/java/org/apache/catalina/ssi/SSIPrintenv.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/ssi/SSIPrintenv.java?rev=646980&r1=646979&r2=646980&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/ssi/SSIPrintenv.java (original)
+++ tomcat/trunk/java/org/apache/catalina/ssi/SSIPrintenv.java Thu Apr 10 14:51:57 2008
@@ -39,10 +39,10 @@
String errorMessage = ssiMediator.getConfigErrMsg();
writer.write(errorMessage);
} else {
- Collection variableNames = ssiMediator.getVariableNames();
- Iterator iter = variableNames.iterator();
+ Collection<String> variableNames = ssiMediator.getVariableNames();
+ Iterator<String> iter = variableNames.iterator();
while (iter.hasNext()) {
- String variableName = (String)iter.next();
+ String variableName = iter.next();
String variableValue = ssiMediator
.getVariableValue(variableName);
//This shouldn't happen, since all the variable names must
Modified: tomcat/trunk/java/org/apache/catalina/ssi/SSIProcessor.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/ssi/SSIProcessor.java?rev=646980&r1=646979&r2=646980&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/ssi/SSIProcessor.java (original)
+++ tomcat/trunk/java/org/apache/catalina/ssi/SSIProcessor.java Thu Apr 10 14:51:57 2008
@@ -40,7 +40,8 @@
protected final static String COMMAND_END = "-->";
protected final static int BUFFER_SIZE = 4096;
protected SSIExternalResolver ssiExternalResolver;
- protected HashMap commands = new HashMap();
+ protected HashMap<String,SSICommand> commands =
+ new HashMap<String,SSICommand>();
protected int debug;
@@ -132,8 +133,8 @@
// change
// during the loop
String configErrMsg = ssiMediator.getConfigErrMsg();
- SSICommand ssiCommand = (SSICommand)commands
- .get(strCmd.toLowerCase());
+ SSICommand ssiCommand =
+ commands.get(strCmd.toLowerCase());
String errorMessage = null;
if (ssiCommand == null) {
errorMessage = "Unknown command: " + strCmd;
Modified: tomcat/trunk/java/org/apache/catalina/ssi/SSIServletExternalResolver.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/ssi/SSIServletExternalResolver.java?rev=646980&r1=646979&r2=646980&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/ssi/SSIServletExternalResolver.java (original)
+++ tomcat/trunk/java/org/apache/catalina/ssi/SSIServletExternalResolver.java Thu Apr 10 14:51:57 2008
@@ -82,7 +82,7 @@
}
- public void addVariableNames(Collection variableNames) {
+ public void addVariableNames(Collection<String> variableNames) {
for (int i = 0; i < VARIABLE_NAMES.length; i++) {
String variableName = VARIABLE_NAMES[i];
String variableValue = getVariableValue(variableName);
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org