Author: markt
Date: Thu Nov 27 13:54:36 2008
New Revision: 721289

URL: http://svn.apache.org/viewvc?rev=721289&view=rev
Log:
Generics changes for remainder of Jasper.

Modified:
    tomcat/trunk/java/org/apache/jasper/runtime/HttpJspBase.java
    tomcat/trunk/java/org/apache/jasper/runtime/JspApplicationContextImpl.java
    tomcat/trunk/java/org/apache/jasper/runtime/JspContextWrapper.java
    tomcat/trunk/java/org/apache/jasper/runtime/JspFactoryImpl.java
    tomcat/trunk/java/org/apache/jasper/runtime/JspRuntimeLibrary.java
    tomcat/trunk/java/org/apache/jasper/runtime/JspWriterImpl.java
    tomcat/trunk/java/org/apache/jasper/runtime/PageContextImpl.java
    tomcat/trunk/java/org/apache/jasper/runtime/PerThreadTagHandlerPool.java
    tomcat/trunk/java/org/apache/jasper/runtime/ProtectedFunctionMapper.java
    tomcat/trunk/java/org/apache/jasper/runtime/TagHandlerPool.java
    tomcat/trunk/java/org/apache/jasper/servlet/JasperLoader.java
    tomcat/trunk/java/org/apache/jasper/servlet/JspCServletContext.java
    tomcat/trunk/java/org/apache/jasper/servlet/JspServlet.java
    tomcat/trunk/java/org/apache/jasper/servlet/JspServletWrapper.java
    tomcat/trunk/java/org/apache/jasper/util/Enumerator.java

Modified: tomcat/trunk/java/org/apache/jasper/runtime/HttpJspBase.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/jasper/runtime/HttpJspBase.java?rev=721289&r1=721288&r2=721289&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/jasper/runtime/HttpJspBase.java (original)
+++ tomcat/trunk/java/org/apache/jasper/runtime/HttpJspBase.java Thu Nov 27 
13:54:36 2008
@@ -25,7 +25,6 @@
 import javax.servlet.http.HttpServletRequest;
 import javax.servlet.http.HttpServletResponse;
 import javax.servlet.jsp.HttpJspPage;
-import javax.servlet.jsp.JspFactory;
 
 import org.apache.jasper.compiler.Localizer;
 

Modified: 
tomcat/trunk/java/org/apache/jasper/runtime/JspApplicationContextImpl.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/jasper/runtime/JspApplicationContextImpl.java?rev=721289&r1=721288&r2=721289&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/jasper/runtime/JspApplicationContextImpl.java 
(original)
+++ tomcat/trunk/java/org/apache/jasper/runtime/JspApplicationContextImpl.java 
Thu Nov 27 13:54:36 2008
@@ -106,8 +106,9 @@
                if (this.resolver == null) {
                        CompositeELResolver r = new CompositeELResolver();
                        r.add(new ImplicitObjectELResolver());
-                       for (Iterator itr = this.resolvers.iterator(); 
itr.hasNext();) {
-                               r.add((ELResolver) itr.next());
+                       for (Iterator<ELResolver> itr = 
this.resolvers.iterator();
+                               itr.hasNext();) {
+                               r.add(itr.next());
                        }
                        r.add(new MapELResolver());
                        r.add(new ResourceBundleELResolver());

Modified: tomcat/trunk/java/org/apache/jasper/runtime/JspContextWrapper.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/jasper/runtime/JspContextWrapper.java?rev=721289&r1=721288&r2=721289&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/jasper/runtime/JspContextWrapper.java 
(original)
+++ tomcat/trunk/java/org/apache/jasper/runtime/JspContextWrapper.java Thu Nov 
27 13:54:36 2008
@@ -65,20 +65,21 @@
        private transient HashMap<String, Object> pageAttributes;
 
        // ArrayList of NESTED scripting variables
-       private ArrayList nestedVars;
+       private ArrayList<String> nestedVars;
 
        // ArrayList of AT_BEGIN scripting variables
-       private ArrayList atBeginVars;
+       private ArrayList<String> atBeginVars;
 
        // ArrayList of AT_END scripting variables
-       private ArrayList atEndVars;
+       private ArrayList<String> atEndVars;
 
-       private Map aliases;
+       private Map<String,String> aliases;
 
        private HashMap<String, Object> originalNestedVars;
 
-       public JspContextWrapper(JspContext jspContext, ArrayList nestedVars,
-                       ArrayList atBeginVars, ArrayList atEndVars, Map 
aliases) {
+       public JspContextWrapper(JspContext jspContext,
+               ArrayList<String> nestedVars, ArrayList<String> atBeginVars,
+               ArrayList<String> atEndVars, Map<String,String> aliases) {
                this.invokingJspCtxt = (PageContext) jspContext;
                this.nestedVars = nestedVars;
                this.atBeginVars = atBeginVars;
@@ -222,7 +223,7 @@
 
        public Enumeration<String> getAttributeNamesInScope(int scope) {
                if (scope == PAGE_SCOPE) {
-                       return new 
Enumerator(pageAttributes.keySet().iterator());
+                       return new 
Enumerator<String>(pageAttributes.keySet().iterator());
                }
 
                return invokingJspCtxt.getAttributeNamesInScope(scope);
@@ -351,7 +352,7 @@
         *            variable scope (one of NESTED, AT_BEGIN, or AT_END)
         */
        private void copyTagToPageScope(int scope) {
-               Iterator iter = null;
+               Iterator<String> iter = null;
 
                switch (scope) {
                case VariableInfo.NESTED:
@@ -372,7 +373,7 @@
                }
 
                while ((iter != null) && iter.hasNext()) {
-                       String varName = (String) iter.next();
+                       String varName = iter.next();
                        Object obj = getAttribute(varName);
                        varName = findAlias(varName);
                        if (obj != null) {
@@ -389,9 +390,9 @@
         */
        private void saveNestedVariables() {
                if (nestedVars != null) {
-                       Iterator iter = nestedVars.iterator();
+                       Iterator<String> iter = nestedVars.iterator();
                        while (iter.hasNext()) {
-                               String varName = (String) iter.next();
+                               String varName = iter.next();
                                varName = findAlias(varName);
                                Object obj = 
invokingJspCtxt.getAttribute(varName);
                                if (obj != null) {
@@ -406,9 +407,9 @@
         */
        private void restoreNestedVariables() {
                if (nestedVars != null) {
-                       Iterator iter = nestedVars.iterator();
+                       Iterator<String> iter = nestedVars.iterator();
                        while (iter.hasNext()) {
-                               String varName = (String) iter.next();
+                               String varName = iter.next();
                                varName = findAlias(varName);
                                Object obj = originalNestedVars.get(varName);
                                if (obj != null) {
@@ -434,7 +435,7 @@
                if (aliases == null)
                        return varName;
 
-               String alias = (String) aliases.get(varName);
+               String alias = aliases.get(varName);
                if (alias == null) {
                        return varName;
                }

Modified: tomcat/trunk/java/org/apache/jasper/runtime/JspFactoryImpl.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/jasper/runtime/JspFactoryImpl.java?rev=721289&r1=721288&r2=721289&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/jasper/runtime/JspFactoryImpl.java (original)
+++ tomcat/trunk/java/org/apache/jasper/runtime/JspFactoryImpl.java Thu Nov 27 
13:54:36 2008
@@ -56,9 +56,9 @@
 
         if( Constants.IS_SECURITY_ENABLED ) {
             PrivilegedGetPageContext dp = new PrivilegedGetPageContext(
-                    (JspFactoryImpl)this, servlet, request, response, 
errorPageURL,
+                    this, servlet, request, response, errorPageURL,
                     needsSession, bufferSize, autoflush);
-            return (PageContext)AccessController.doPrivileged(dp);
+            return AccessController.doPrivileged(dp);
         } else {
             return internalGetPageContext(servlet, request, response,
                     errorPageURL, needsSession,
@@ -71,7 +71,7 @@
             return;
         if( Constants.IS_SECURITY_ENABLED ) {
             PrivilegedReleasePageContext dp = new PrivilegedReleasePageContext(
-                    (JspFactoryImpl)this,pc);
+                    this,pc);
             AccessController.doPrivileged(dp);
         } else {
             internalReleasePageContext(pc);
@@ -121,7 +121,8 @@
         }
     }
 
-    private class PrivilegedGetPageContext implements PrivilegedAction {
+    private class PrivilegedGetPageContext
+            implements PrivilegedAction<PageContext> {
 
         private JspFactoryImpl factory;
         private Servlet servlet;
@@ -145,13 +146,14 @@
             this.autoflush = autoflush;
         }
 
-        public Object run() {
+        public PageContext run() {
             return factory.internalGetPageContext(servlet, request, response,
                     errorPageURL, needsSession, bufferSize, autoflush);
         }
     }
 
-    private class PrivilegedReleasePageContext implements PrivilegedAction {
+    private class PrivilegedReleasePageContext
+            implements PrivilegedAction<Void> {
 
         private JspFactoryImpl factory;
         private PageContext pageContext;
@@ -162,7 +164,7 @@
             this.pageContext = pageContext;
         }
 
-        public Object run() {
+        public Void run() {
             factory.internalReleasePageContext(pageContext);
             return null;
         }

Modified: tomcat/trunk/java/org/apache/jasper/runtime/JspRuntimeLibrary.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/jasper/runtime/JspRuntimeLibrary.java?rev=721289&r1=721288&r2=721289&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/jasper/runtime/JspRuntimeLibrary.java 
(original)
+++ tomcat/trunk/java/org/apache/jasper/runtime/JspRuntimeLibrary.java Thu Nov 
27 13:54:36 2008
@@ -61,7 +61,7 @@
         = "javax.servlet.jsp.jspException";
 
     protected static class PrivilegedIntrospectHelper
-        implements PrivilegedExceptionAction {
+        implements PrivilegedExceptionAction<Void> {
 
         private Object bean;
         private String prop;
@@ -82,7 +82,7 @@
             this.ignoreMethodNF = ignoreMethodNF;
         }
          
-        public Object run() throws JasperException {
+        public Void run() throws JasperException {
             internalIntrospecthelper(
                 bean,prop,value,request,param,ignoreMethodNF);
             return null;
@@ -175,7 +175,7 @@
             return Long.valueOf(s).longValue();
     }
 
-    public static Object coerce(String s, Class target) {
+    public static Object coerce(String s, Class<?> target) {
 
         boolean isNullOrEmpty = (s == null || s.length() == 0);
 
@@ -225,8 +225,8 @@
     }
 
    // __begin convertMethod
-    public static Object convert(String propertyName, String s, Class t,
-                                 Class propertyEditorClass) 
+    public static Object convert(String propertyName, String s, Class<?> t,
+            Class<?> propertyEditorClass) 
        throws JasperException 
     {
         try {
@@ -279,9 +279,9 @@
     public static void introspect(Object bean, ServletRequest request)
                                   throws JasperException
     {
-        Enumeration e = request.getParameterNames();
+        Enumeration<String> e = request.getParameterNames();
         while ( e.hasMoreElements() ) {
-            String name  = (String) e.nextElement();
+            String name  = e.nextElement();
             String value = request.getParameter(name);
             introspecthelper(bean, name, value, request, name, true);
         }
@@ -316,8 +316,8 @@
                                         throws JasperException
     {
         Method method = null;
-        Class type = null;
-        Class propertyEditorClass = null;
+        Class<?> type = null;
+        Class<?> propertyEditorClass = null;
         try {
             java.beans.BeanInfo info
                 = java.beans.Introspector.getBeanInfo(bean.getClass());
@@ -339,7 +339,7 @@
                         throw new JasperException(
                             
Localizer.getMessage("jsp.error.beans.setproperty.noindexset"));
                     }
-                    Class t = type.getComponentType();
+                    Class<?> t = type.getComponentType();
                     String[] values = request.getParameterValues(param);
                     //XXX Please check.
                     if(values == null) return;
@@ -428,8 +428,8 @@
                                         Object bean,
                                         Method method,
                                         String[] values,
-                                        Class t,
-                                        Class propertyEditorClass)
+                                        Class<?> t,
+                                        Class<?> propertyEditorClass)
                 throws JasperException {
 
         try {
@@ -764,10 +764,10 @@
         }        
     }
     
-    public static Method getWriteMethod(Class beanClass, String prop)
+    public static Method getWriteMethod(Class<?> beanClass, String prop)
     throws JasperException {
         Method method = null;        
-        Class type = null;
+        Class<?> type = null;
         try {
             java.beans.BeanInfo info
                 = java.beans.Introspector.getBeanInfo(beanClass);
@@ -807,11 +807,11 @@
         return method;
     }
 
-    public static Method getReadMethod(Class beanClass, String prop)
+    public static Method getReadMethod(Class<?> beanClass, String prop)
             throws JasperException {
 
         Method method = null;        
-        Class type = null;
+        Class<?> type = null;
         try {
             java.beans.BeanInfo info
                 = java.beans.Introspector.getBeanInfo(beanClass);
@@ -853,12 +853,13 @@
     // PropertyEditor Support
 
     public static Object getValueFromBeanInfoPropertyEditor(
-                           Class attrClass, String attrName, String attrValue,
-                           Class propertyEditorClass) 
+                           Class<?> attrClass, String attrName, String 
attrValue,
+                           Class<?> propertyEditorClass) 
         throws JasperException 
     {
         try {
-            PropertyEditor pe = 
(PropertyEditor)propertyEditorClass.newInstance();
+            PropertyEditor pe =
+                (PropertyEditor)propertyEditorClass.newInstance();
             pe.setAsText(attrValue);
             return pe.getValue();
         } catch (Exception ex) {
@@ -870,7 +871,7 @@
     }
 
     public static Object getValueFromPropertyEditorManager(
-                     Class attrClass, String attrName, String attrValue) 
+                     Class<?> attrClass, String attrName, String attrValue) 
         throws JasperException 
     {
         try {

Modified: tomcat/trunk/java/org/apache/jasper/runtime/JspWriterImpl.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/jasper/runtime/JspWriterImpl.java?rev=721289&r1=721288&r2=721289&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/jasper/runtime/JspWriterImpl.java (original)
+++ tomcat/trunk/java/org/apache/jasper/runtime/JspWriterImpl.java Thu Nov 27 
13:54:36 2008
@@ -128,8 +128,8 @@
     
     private String getLocalizeMessage(final String message){
         if (SecurityUtil.isPackageProtectionEnabled()){
-            return (String)AccessController.doPrivileged(new 
PrivilegedAction(){
-                public Object run(){
+            return AccessController.doPrivileged(new 
PrivilegedAction<String>(){
+                public String run(){
                     return Localizer.getMessage(message); 
                 }
             });

Modified: tomcat/trunk/java/org/apache/jasper/runtime/PageContextImpl.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/jasper/runtime/PageContextImpl.java?rev=721289&r1=721288&r2=721289&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/jasper/runtime/PageContextImpl.java (original)
+++ tomcat/trunk/java/org/apache/jasper/runtime/PageContextImpl.java Thu Nov 27 
13:54:36 2008
@@ -214,7 +214,8 @@
                }
 
                if (SecurityUtil.isPackageProtectionEnabled()) {
-                       return AccessController.doPrivileged(new 
PrivilegedAction() {
+                       return AccessController.doPrivileged(
+                               new PrivilegedAction<Object>() {
                                public Object run() {
                                        return doGetAttribute(name);
                                }
@@ -237,7 +238,8 @@
                }
 
                if (SecurityUtil.isPackageProtectionEnabled()) {
-                       return AccessController.doPrivileged(new 
PrivilegedAction() {
+                       return AccessController.doPrivileged(
+                               new PrivilegedAction<Object>() {
                                public Object run() {
                                        return doGetAttribute(name, scope);
                                }
@@ -279,8 +281,8 @@
                }
 
                if (SecurityUtil.isPackageProtectionEnabled()) {
-                       AccessController.doPrivileged(new PrivilegedAction() {
-                               public Object run() {
+                       AccessController.doPrivileged(new 
PrivilegedAction<Void>() {
+                               public Void run() {
                                        doSetAttribute(name, attribute);
                                        return null;
                                }
@@ -306,8 +308,8 @@
                }
 
                if (SecurityUtil.isPackageProtectionEnabled()) {
-                       AccessController.doPrivileged(new PrivilegedAction() {
-                               public Object run() {
+                       AccessController.doPrivileged(new 
PrivilegedAction<Void>() {
+                               public Void run() {
                                        doSetAttribute(name, o, scope);
                                        return null;
                                }
@@ -356,8 +358,8 @@
                                        
.getMessage("jsp.error.attribute.null_name"));
                }
                if (SecurityUtil.isPackageProtectionEnabled()) {
-                       AccessController.doPrivileged(new PrivilegedAction() {
-                               public Object run() {
+                       AccessController.doPrivileged(new 
PrivilegedAction<Void>() {
+                               public Void run() {
                                        doRemoveAttribute(name, scope);
                                        return null;
                                }
@@ -402,9 +404,9 @@
                }
 
                if (SecurityUtil.isPackageProtectionEnabled()) {
-                       return ((Integer) AccessController
-                                       .doPrivileged(new PrivilegedAction() {
-                                               public Object run() {
+                       return (AccessController
+                                       .doPrivileged(new 
PrivilegedAction<Integer>() {
+                                               public Integer run() {
                                                        return new 
Integer(doGetAttributeScope(name));
                                                }
                                        })).intValue();
@@ -433,7 +435,8 @@
 
        public Object findAttribute(final String name) {
                if (SecurityUtil.isPackageProtectionEnabled()) {
-                       return AccessController.doPrivileged(new 
PrivilegedAction() {
+                       return AccessController.doPrivileged(
+                               new PrivilegedAction<Object>() {
                                public Object run() {
                                        if (name == null) {
                                                throw new 
NullPointerException(Localizer
@@ -474,9 +477,9 @@
 
        public Enumeration<String> getAttributeNamesInScope(final int scope) {
                if (SecurityUtil.isPackageProtectionEnabled()) {
-                       return (Enumeration) AccessController
-                                       .doPrivileged(new PrivilegedAction() {
-                                               public Object run() {
+                       return AccessController.doPrivileged(
+                               new PrivilegedAction<Enumeration<String>>() {
+                                               public Enumeration<String> 
run() {
                                                        return 
doGetAttributeNamesInScope(scope);
                                                }
                                        });
@@ -485,10 +488,10 @@
                }
        }
 
-       private Enumeration doGetAttributeNamesInScope(int scope) {
+       private Enumeration<String> doGetAttributeNamesInScope(int scope) {
                switch (scope) {
                case PAGE_SCOPE:
-                       return new Enumerator(attributes.keySet().iterator());
+                       return new 
Enumerator<String>(attributes.keySet().iterator());
 
                case REQUEST_SCOPE:
                        return request.getAttributeNames();
@@ -516,8 +519,8 @@
                }
 
                if (SecurityUtil.isPackageProtectionEnabled()) {
-                       AccessController.doPrivileged(new PrivilegedAction() {
-                               public Object run() {
+                       AccessController.doPrivileged(new 
PrivilegedAction<Void>() {
+                               public Void run() {
                                        doRemoveAttribute(name);
                                        return null;
                                }
@@ -616,8 +619,9 @@
                        throws ServletException, IOException {
                if (SecurityUtil.isPackageProtectionEnabled()) {
                        try {
-                               AccessController.doPrivileged(new 
PrivilegedExceptionAction() {
-                                       public Object run() throws Exception {
+                               AccessController.doPrivileged(
+                                       new PrivilegedExceptionAction<Void>() {
+                                       public Void run() throws Exception {
                                                doInclude(relativeUrlPath, 
flush);
                                                return null;
                                        }
@@ -649,8 +653,9 @@
                        IOException {
                if (SecurityUtil.isPackageProtectionEnabled()) {
                        try {
-                               AccessController.doPrivileged(new 
PrivilegedExceptionAction() {
-                                       public Object run() throws Exception {
+                               AccessController.doPrivileged(
+                                       new PrivilegedExceptionAction<Void>() {
+                                       public Void run() throws Exception {
                                                doForward(relativeUrlPath);
                                                return null;
                                        }
@@ -763,8 +768,9 @@
 
                if (SecurityUtil.isPackageProtectionEnabled()) {
                        try {
-                               AccessController.doPrivileged(new 
PrivilegedExceptionAction() {
-                                       public Object run() throws Exception {
+                               AccessController.doPrivileged(
+                                       new PrivilegedExceptionAction<Void>() {
+                                       public Void run() throws Exception {
                                                doHandlePageException(t);
                                                return null;
                                        }
@@ -893,7 +899,7 @@
         * @return The result of the evaluation
         */
        public static Object proprietaryEvaluate(final String expression,
-                       final Class expectedType, final PageContext pageContext,
+                       final Class<?> expectedType, final PageContext 
pageContext,
                        final ProtectedFunctionMapper functionMap, final 
boolean escape)
                        throws ELException {
                Object retValue;
@@ -901,7 +907,7 @@
                if (SecurityUtil.isPackageProtectionEnabled()) {
                        try {
                                retValue = AccessController
-                                               .doPrivileged(new 
PrivilegedExceptionAction() {
+                                               .doPrivileged(new 
PrivilegedExceptionAction<Object>() {
 
                                                        public Object run() 
throws Exception {
                                 ELContextImpl ctx = (ELContextImpl) 
pageContext.getELContext();

Modified: 
tomcat/trunk/java/org/apache/jasper/runtime/PerThreadTagHandlerPool.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/jasper/runtime/PerThreadTagHandlerPool.java?rev=721289&r1=721288&r2=721289&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/jasper/runtime/PerThreadTagHandlerPool.java 
(original)
+++ tomcat/trunk/java/org/apache/jasper/runtime/PerThreadTagHandlerPool.java 
Thu Nov 27 13:54:36 2008
@@ -37,9 +37,9 @@
     private int maxSize;
 
     // For cleanup
-    private Vector perThreadDataVector;
+    private Vector<PerThreadData> perThreadDataVector;
 
-    private ThreadLocal perThread;
+    private ThreadLocal<PerThreadData> perThread;
 
     private static class PerThreadData {
         Tag handlers[];
@@ -51,7 +51,7 @@
      */
     public PerThreadTagHandlerPool() {
         super();
-        perThreadDataVector = new Vector();
+        perThreadDataVector = new Vector<PerThreadData>();
     }
 
     protected void init(ServletConfig config) {
@@ -64,8 +64,8 @@
             }
         }
 
-        perThread = new ThreadLocal() {
-            protected Object initialValue() {
+        perThread = new ThreadLocal<PerThreadData>() {
+            protected PerThreadData initialValue() {
                 PerThreadData ptd = new PerThreadData();
                 ptd.handlers = new Tag[maxSize];
                 ptd.current = -1;
@@ -85,13 +85,13 @@
      *
      * @throws JspException if a tag handler cannot be instantiated
      */
-    public Tag get(Class handlerClass) throws JspException {
-        PerThreadData ptd = (PerThreadData)perThread.get();
+    public Tag get(Class<? extends Tag> handlerClass) throws JspException {
+        PerThreadData ptd = perThread.get();
         if(ptd.current >=0 ) {
             return ptd.handlers[ptd.current--];
         } else {
            try {
-               return (Tag) handlerClass.newInstance();
+               return handlerClass.newInstance();
            } catch (Exception e) {
                throw new JspException(e.getMessage(), e);
            }
@@ -106,7 +106,7 @@
      * @param handler Tag handler to add to this tag handler pool
      */
     public void reuse(Tag handler) {
-        PerThreadData ptd=(PerThreadData)perThread.get();
+        PerThreadData ptd = perThread.get();
        if (ptd.current < (ptd.handlers.length - 1)) {
            ptd.handlers[++ptd.current] = handler;
         } else {
@@ -118,9 +118,9 @@
      * Calls the release() method of all tag handlers in this tag handler pool.
      */
     public void release() {        
-        Enumeration enumeration = perThreadDataVector.elements();
+        Enumeration<PerThreadData> enumeration = 
perThreadDataVector.elements();
         while (enumeration.hasMoreElements()) {
-           PerThreadData ptd = (PerThreadData)enumeration.nextElement();
+           PerThreadData ptd = enumeration.nextElement();
             if (ptd.handlers != null) {
                 for (int i=ptd.current; i>=0; i--) {
                     if (ptd.handlers[i] != null) {

Modified: 
tomcat/trunk/java/org/apache/jasper/runtime/ProtectedFunctionMapper.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/jasper/runtime/ProtectedFunctionMapper.java?rev=721289&r1=721288&r2=721289&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/jasper/runtime/ProtectedFunctionMapper.java 
(original)
+++ tomcat/trunk/java/org/apache/jasper/runtime/ProtectedFunctionMapper.java 
Thu Nov 27 13:54:36 2008
@@ -40,7 +40,7 @@
     /**
      * Maps "prefix:name" to java.lang.Method objects.
      */
-    private HashMap fnmap = null;
+    private HashMap<String,Method> fnmap = null;
 
     /**
      * If there is only one function in the map, this is the Method for it.
@@ -64,16 +64,16 @@
     public static ProtectedFunctionMapper getInstance() {
         ProtectedFunctionMapper funcMapper;
         if (SecurityUtil.isPackageProtectionEnabled()) {
-            funcMapper = (ProtectedFunctionMapper) AccessController
-                    .doPrivileged(new PrivilegedAction() {
-                        public Object run() {
+            funcMapper = AccessController.doPrivileged(
+                    new PrivilegedAction<ProtectedFunctionMapper>() {
+                        public ProtectedFunctionMapper run() {
                             return new ProtectedFunctionMapper();
                         }
                     });
         } else {
             funcMapper = new ProtectedFunctionMapper();
         }
-        funcMapper.fnmap = new java.util.HashMap();
+        funcMapper.fnmap = new HashMap<String,Method>();
         return funcMapper;
     }
 
@@ -92,15 +92,14 @@
      * @throws RuntimeException
      *             if no method with the given signature could be found.
      */
-    public void mapFunction(String fnQName, final Class c,
-            final String methodName, final Class[] args) {
+    public void mapFunction(String fnQName, final Class<?> c,
+            final String methodName, final Class<?>[] args) {
         java.lang.reflect.Method method;
         if (SecurityUtil.isPackageProtectionEnabled()) {
             try {
-                method = (java.lang.reflect.Method) AccessController
-                        .doPrivileged(new PrivilegedExceptionAction() {
-
-                            public Object run() throws Exception {
+                method = AccessController.doPrivileged(
+                        new PrivilegedExceptionAction<Method>() {
+                            public Method run() throws Exception {
                                 return c.getDeclaredMethod(methodName, args);
                             }
                         });
@@ -139,22 +138,21 @@
      *             if no method with the given signature could be found.
      */
     public static ProtectedFunctionMapper getMapForFunction(String fnQName,
-            final Class c, final String methodName, final Class[] args) {
+            final Class<?> c, final String methodName, final Class<?>[] args) {
         java.lang.reflect.Method method;
         ProtectedFunctionMapper funcMapper;
         if (SecurityUtil.isPackageProtectionEnabled()) {
-            funcMapper = (ProtectedFunctionMapper) AccessController
-                    .doPrivileged(new PrivilegedAction() {
-                        public Object run() {
+            funcMapper = AccessController.doPrivileged(
+                    new PrivilegedAction<ProtectedFunctionMapper>() {
+                        public ProtectedFunctionMapper run() {
                             return new ProtectedFunctionMapper();
                         }
                     });
 
             try {
-                method = (java.lang.reflect.Method) AccessController
-                        .doPrivileged(new PrivilegedExceptionAction() {
-
-                            public Object run() throws Exception {
+                method = AccessController.doPrivileged(
+                        new PrivilegedExceptionAction<Method>() {
+                            public Method run() throws Exception {
                                 return c.getDeclaredMethod(methodName, args);
                             }
                         });
@@ -189,7 +187,7 @@
      */
     public Method resolveFunction(String prefix, String localName) {
         if (this.fnmap != null) {
-            return (Method) this.fnmap.get(prefix + ":" + localName);
+            return this.fnmap.get(prefix + ":" + localName);
         }
         return theMethod;
     }

Modified: tomcat/trunk/java/org/apache/jasper/runtime/TagHandlerPool.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/jasper/runtime/TagHandlerPool.java?rev=721289&r1=721288&r2=721289&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/jasper/runtime/TagHandlerPool.java (original)
+++ tomcat/trunk/java/org/apache/jasper/runtime/TagHandlerPool.java Thu Nov 27 
13:54:36 2008
@@ -50,7 +50,7 @@
         String tpClassName=getOption( config, OPTION_TAGPOOL, null);
         if( tpClassName != null ) {
             try {
-                Class c=Class.forName( tpClassName );
+                Class<?> c=Class.forName( tpClassName );
                 result=(TagHandlerPool)c.newInstance();
             } catch (Exception e) {
                 e.printStackTrace();
@@ -110,7 +110,7 @@
      *
      * @throws JspException if a tag handler cannot be instantiated
      */
-    public Tag get(Class handlerClass) throws JspException {
+    public Tag get(Class<? extends Tag> handlerClass) throws JspException {
        Tag handler;
         synchronized( this ) {
             if (current >= 0) {
@@ -125,7 +125,7 @@
                if (Constants.USE_INSTANCE_MANAGER_FOR_TAGS) {
                        return (Tag) 
instanceManager.newInstance(handlerClass.getName(), 
handlerClass.getClassLoader());
                } else {
-                Tag instance = (Tag) handlerClass.newInstance();
+                Tag instance = handlerClass.newInstance();
                 instanceManager.newInstance(instance);
                 return instance;
                }

Modified: tomcat/trunk/java/org/apache/jasper/servlet/JasperLoader.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/jasper/servlet/JasperLoader.java?rev=721289&r1=721288&r2=721289&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/jasper/servlet/JasperLoader.java (original)
+++ tomcat/trunk/java/org/apache/jasper/servlet/JasperLoader.java Thu Nov 27 
13:54:36 2008
@@ -61,7 +61,7 @@
      *
      * @exception ClassNotFoundException if the class was not found
      */
-    public Class loadClass(String name) throws ClassNotFoundException {
+    public Class<?> loadClass(String name) throws ClassNotFoundException {
 
         return (loadClass(name, false));
     }
@@ -91,10 +91,10 @@
      *                                     
      * @exception ClassNotFoundException if the class was not found
      */                                    
-    public Class loadClass(final String name, boolean resolve)
+    public Class<?> loadClass(final String name, boolean resolve)
         throws ClassNotFoundException {
 
-        Class clazz = null;                
+        Class<?> clazz = null;                
                                            
         // (0) Check our previously loaded class cache
         clazz = findLoadedClass(name);     

Modified: tomcat/trunk/java/org/apache/jasper/servlet/JspCServletContext.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/jasper/servlet/JspCServletContext.java?rev=721289&r1=721288&r2=721289&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/jasper/servlet/JspCServletContext.java 
(original)
+++ tomcat/trunk/java/org/apache/jasper/servlet/JspCServletContext.java Thu Nov 
27 13:54:36 2008
@@ -51,7 +51,7 @@
     /**
      * Servlet context attributes.
      */
-    protected Hashtable myAttributes;
+    protected Hashtable<String,Object> myAttributes;
 
 
     /**
@@ -77,7 +77,7 @@
      */
     public JspCServletContext(PrintWriter aLogWriter, URL aResourceBaseURL) {
 
-        myAttributes = new Hashtable();
+        myAttributes = new Hashtable<String,Object>();
         myLogWriter = aLogWriter;
         myResourceBaseURL = aResourceBaseURL;
 
@@ -102,7 +102,7 @@
     /**
      * Return an enumeration of context attribute names.
      */
-    public Enumeration getAttributeNames() {
+    public Enumeration<String> getAttributeNames() {
 
         return (myAttributes.keys());
 
@@ -147,9 +147,9 @@
      * Return an enumeration of the names of context initialization
      * parameters.
      */
-    public Enumeration getInitParameterNames() {
+    public Enumeration<String> getInitParameterNames() {
 
-        return (new Vector().elements());
+        return (new Vector<String>().elements());
 
     }
 
@@ -289,9 +289,9 @@
      *
      * @param path Context-relative base path
      */
-    public Set getResourcePaths(String path) {
+    public Set<String> getResourcePaths(String path) {
 
-        Set thePaths = new HashSet();
+        Set<String> thePaths = new HashSet<String>();
         if (!path.endsWith("/"))
             path += "/";
         String basePath = getRealPath(path);
@@ -352,9 +352,9 @@
      *
      * @deprecated This method has been deprecated with no replacement
      */
-    public Enumeration getServletNames() {
+    public Enumeration<String> getServletNames() {
 
-        return (new Vector().elements());
+        return (new Vector<String>().elements());
 
     }
 
@@ -364,9 +364,9 @@
      *
      * @deprecated This method has been deprecated with no replacement
      */
-    public Enumeration getServlets() {
+    public Enumeration<String> getServlets() {
 
-        return (new Vector().elements());
+        return (new Vector<String>().elements());
 
     }
 

Modified: tomcat/trunk/java/org/apache/jasper/servlet/JspServlet.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/jasper/servlet/JspServlet.java?rev=721289&r1=721288&r2=721289&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/jasper/servlet/JspServlet.java (original)
+++ tomcat/trunk/java/org/apache/jasper/servlet/JspServlet.java Thu Nov 27 
13:54:36 2008
@@ -85,9 +85,12 @@
             try {
                 ClassLoader loader = Thread.currentThread()
                         .getContextClassLoader();
-                Class engineOptionsClass = loader.loadClass(engineOptionsName);
-                Class[] ctorSig = { ServletConfig.class, ServletContext.class 
};
-                Constructor ctor = engineOptionsClass.getConstructor(ctorSig);
+                Class<?> engineOptionsClass =
+                    loader.loadClass(engineOptionsName);
+                Class<?>[] ctorSig =
+                    { ServletConfig.class, ServletContext.class };
+                Constructor<?> ctor =
+                    engineOptionsClass.getConstructor(ctorSig);
                 Object[] args = { config, context };
                 options = (Options) ctor.newInstance(args);
             } catch (Throwable e) {
@@ -255,9 +258,9 @@
             log.debug("\t      RequestURI: " + request.getRequestURI());
             log.debug("\t     QueryString: " + request.getQueryString());
             log.debug("\t  Request Params: ");
-            Enumeration e = request.getParameterNames();
+            Enumeration<String> e = request.getParameterNames();
             while (e.hasMoreElements()) {
-                String name = (String) e.nextElement();
+                String name = e.nextElement();
                 log.debug("\t\t " + name + " = " 
                           + request.getParameter(name));
             }

Modified: tomcat/trunk/java/org/apache/jasper/servlet/JspServletWrapper.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/jasper/servlet/JspServletWrapper.java?rev=721289&r1=721288&r2=721289&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/jasper/servlet/JspServletWrapper.java 
(original)
+++ tomcat/trunk/java/org/apache/jasper/servlet/JspServletWrapper.java Thu Nov 
27 13:54:36 2008
@@ -69,7 +69,7 @@
 
     private Servlet theServlet;
     private String jspUri;
-    private Class tagHandlerClass;
+    private Class<?> tagHandlerClass;
     private JspCompilationContext ctxt;
     private long available = 0L;
     private ServletConfig config;
@@ -198,7 +198,7 @@
     /**
      * Compile (if needed) and load a tag file
      */
-    public Class loadTagFile() throws JasperException {
+    public Class<?> loadTagFile() throws JasperException {
 
         try {
             if (ctxt.isRemoved()) {
@@ -232,7 +232,7 @@
      * (skeleton) with no dependencies on other other tag files is
      * generated and compiled.
      */
-    public Class loadTagFilePrototype() throws JasperException {
+    public Class<?> loadTagFilePrototype() throws JasperException {
 
         ctxt.setPrototypeMode(true);
         try {
@@ -258,7 +258,7 @@
                 target = getServlet();
             }
             if (target != null && target instanceof JspSourceDependent) {
-                return ((java.util.List) ((JspSourceDependent) 
target).getDependants());
+                return ((JspSourceDependent) target).getDependants();
             }
         } catch (Throwable ex) {
         }

Modified: tomcat/trunk/java/org/apache/jasper/util/Enumerator.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/jasper/util/Enumerator.java?rev=721289&r1=721288&r2=721289&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/jasper/util/Enumerator.java (original)
+++ tomcat/trunk/java/org/apache/jasper/util/Enumerator.java Thu Nov 27 
13:54:36 2008
@@ -38,7 +38,7 @@
  * @version $Revision$ $Date$
  */
 
-public final class Enumerator implements Enumeration {
+public final class Enumerator<T> implements Enumeration<T> {
 
 
     // ----------------------------------------------------------- Constructors
@@ -49,7 +49,7 @@
      *
      * @param collection Collection whose values should be enumerated
      */
-    public Enumerator(Collection collection) {
+    public Enumerator(Collection<T> collection) {
 
         this(collection.iterator());
 
@@ -62,7 +62,7 @@
      * @param collection Collection whose values should be enumerated
      * @param clone true to clone iterator
      */
-    public Enumerator(Collection collection, boolean clone) {
+    public Enumerator(Collection<T> collection, boolean clone) {
 
         this(collection.iterator(), clone);
 
@@ -75,7 +75,7 @@
      *
      * @param iterator Iterator to be wrapped
      */
-    public Enumerator(Iterator iterator) {
+    public Enumerator(Iterator<T> iterator) {
 
         super();
         this.iterator = iterator;
@@ -90,13 +90,13 @@
      * @param iterator Iterator to be wrapped
      * @param clone true to clone iterator
      */
-    public Enumerator(Iterator iterator, boolean clone) {
+    public Enumerator(Iterator<T> iterator, boolean clone) {
 
         super();
         if (!clone) {
             this.iterator = iterator;
         } else {
-            List list = new ArrayList();
+            List<T> list = new ArrayList<T>();
             while (iterator.hasNext()) {
                 list.add(iterator.next());
             }
@@ -111,7 +111,7 @@
      *
      * @param map Map whose values should be enumerated
      */
-    public Enumerator(Map map) {
+    public Enumerator(Map<?,T> map) {
 
         this(map.values().iterator());
 
@@ -124,7 +124,7 @@
      * @param map Map whose values should be enumerated
      * @param clone true to clone iterator
      */
-    public Enumerator(Map map, boolean clone) {
+    public Enumerator(Map<?,T> map, boolean clone) {
 
         this(map.values().iterator(), clone);
 
@@ -138,7 +138,7 @@
      * The <code>Iterator</code> over which the <code>Enumeration</code>
      * represented by this class actually operates.
      */
-    private Iterator iterator = null;
+    private Iterator<T> iterator = null;
 
 
     // --------------------------------------------------------- Public Methods
@@ -166,7 +166,7 @@
      *
      * @exception NoSuchElementException if no more elements exist
      */
-    public Object nextElement() throws NoSuchElementException {
+    public T nextElement() throws NoSuchElementException {
 
         return (iterator.next());
 



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

Reply via email to