Author: simonetripodi
Date: Sun Feb 20 15:40:57 2011
New Revision: 1072614

URL: http://svn.apache.org/viewvc?rev=1072614&view=rev
Log:
stripped out the TypeConverter interface, better not reiventing the wheel

Removed:
    
commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/rulesbinder/ConverterBuilder.java
    
commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/spi/TypeConverter.java
Modified:
    
commons/sandbox/digester3/trunk/src/examples/document-markup/src/main/java/org/apache/commons/digester3/examples/documentmarkup/MarkupDigester.java
    
commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/AbstractRulesModule.java
    
commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/Digester.java
    
commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/RulesBinder.java
    
commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/annotations/WithMemoryRulesBinder.java
    
commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/internal/DigesterImpl.java
    
commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/internal/rulesbinder/RulesBinderImpl.java
    
commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/xmlrules/metaparser/PrefixedRulesBinder.java
    
commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/xmlrules/metaparser/WithMemoryRulesBinder.java

Modified: 
commons/sandbox/digester3/trunk/src/examples/document-markup/src/main/java/org/apache/commons/digester3/examples/documentmarkup/MarkupDigester.java
URL: 
http://svn.apache.org/viewvc/commons/sandbox/digester3/trunk/src/examples/document-markup/src/main/java/org/apache/commons/digester3/examples/documentmarkup/MarkupDigester.java?rev=1072614&r1=1072613&r2=1072614&view=diff
==============================================================================
--- 
commons/sandbox/digester3/trunk/src/examples/document-markup/src/main/java/org/apache/commons/digester3/examples/documentmarkup/MarkupDigester.java
 (original)
+++ 
commons/sandbox/digester3/trunk/src/examples/document-markup/src/main/java/org/apache/commons/digester3/examples/documentmarkup/MarkupDigester.java
 Sun Feb 20 15:40:57 2011
@@ -28,7 +28,6 @@ import java.util.Stack;
 import org.apache.commons.digester3.Digester;
 import org.apache.commons.digester3.Rule;
 import org.apache.commons.digester3.spi.Rules;
-import org.apache.commons.digester3.spi.TypeConverter;
 import org.apache.commons.logging.Log;
 import org.xml.sax.Attributes;
 import org.xml.sax.ErrorHandler;
@@ -284,11 +283,6 @@ public class MarkupDigester implements D
         return this.wrapped.createSAXException(message);
     }
 
-    public <T> TypeConverter<T> lookupConverter(Class<T> type) {
-        // TODO Auto-generated method stub
-        return null;
-    }
-
     /**
      * Iterate over the list of rules most recently matched, and
      * if any of them implement the TextSegmentHandler interface then

Modified: 
commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/AbstractRulesModule.java
URL: 
http://svn.apache.org/viewvc/commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/AbstractRulesModule.java?rev=1072614&r1=1072613&r2=1072614&view=diff
==============================================================================
--- 
commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/AbstractRulesModule.java
 (original)
+++ 
commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/AbstractRulesModule.java
 Sun Feb 20 15:40:57 2011
@@ -17,7 +17,6 @@
  */
 package org.apache.commons.digester3;
 
-import org.apache.commons.digester3.rulesbinder.ConverterBuilder;
 import org.apache.commons.digester3.rulesbinder.LinkedRuleBuilder;
 
 /**
@@ -58,16 +57,6 @@ public abstract class AbstractRulesModul
     }
 
     /**
-     * @see RulesBinder#convert(Class)
-     * @param <T>
-     * @param type
-     * @return
-     */
-    protected <T> ConverterBuilder<T> convert(Class<T> type) {
-        return this.rulesBinder.convert(type);
-    }
-
-    /**
      * Return the wrapped {@link RulesBinder}.
      *
      * @return The wrapped {@link RulesBinder}

Modified: 
commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/Digester.java
URL: 
http://svn.apache.org/viewvc/commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/Digester.java?rev=1072614&r1=1072613&r2=1072614&view=diff
==============================================================================
--- 
commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/Digester.java
 (original)
+++ 
commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/Digester.java
 Sun Feb 20 15:40:57 2011
@@ -26,7 +26,6 @@ import java.util.EmptyStackException;
 import java.util.Map;
 
 import org.apache.commons.digester3.spi.Rules;
-import org.apache.commons.digester3.spi.TypeConverter;
 import org.apache.commons.logging.Log;
 import org.xml.sax.ContentHandler;
 import org.xml.sax.DTDHandler;
@@ -376,15 +375,4 @@ public interface Digester extends Conten
      */
     SAXException createSAXException(String message);
 
-    /**
-     * Allows retrieve the bound converter for the given type.
-     *
-     * NOTE: the first version will be a commons-beanutils Converter facade.
-     *
-     * @param <T>
-     * @param type
-     * @return
-     */
-    <T> TypeConverter<T> lookupConverter(Class<T> type);
-
 }

Modified: 
commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/RulesBinder.java
URL: 
http://svn.apache.org/viewvc/commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/RulesBinder.java?rev=1072614&r1=1072613&r2=1072614&view=diff
==============================================================================
--- 
commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/RulesBinder.java
 (original)
+++ 
commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/RulesBinder.java
 Sun Feb 20 15:40:57 2011
@@ -17,7 +17,6 @@
  */
 package org.apache.commons.digester3;
 
-import org.apache.commons.digester3.rulesbinder.ConverterBuilder;
 import org.apache.commons.digester3.rulesbinder.LinkedRuleBuilder;
 
 /**
@@ -63,15 +62,6 @@ public interface RulesBinder {
     LinkedRuleBuilder forPattern(String pattern);
 
     /**
-     * Allows to associate the given type to one {@link 
org.apache.commons.digester3.spi.TypeConverter}.
-     *
-     * @param <T>
-     * @param type
-     * @return
-     */
-    <T> ConverterBuilder<T> convert(Class<T> type);
-
-    /**
      * Returns the {@code ClassLoader} used in the current bindings.
      *
      * @return The {@code ClassLoader} used in the current bindings.

Modified: 
commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/annotations/WithMemoryRulesBinder.java
URL: 
http://svn.apache.org/viewvc/commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/annotations/WithMemoryRulesBinder.java?rev=1072614&r1=1072613&r2=1072614&view=diff
==============================================================================
--- 
commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/annotations/WithMemoryRulesBinder.java
 (original)
+++ 
commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/annotations/WithMemoryRulesBinder.java
 Sun Feb 20 15:40:57 2011
@@ -22,7 +22,6 @@ import java.util.Set;
 
 import org.apache.commons.digester3.RulesBinder;
 import org.apache.commons.digester3.RulesModule;
-import org.apache.commons.digester3.rulesbinder.ConverterBuilder;
 import org.apache.commons.digester3.rulesbinder.LinkedRuleBuilder;
 
 final class WithMemoryRulesBinder implements RulesBinder {
@@ -74,13 +73,6 @@ final class WithMemoryRulesBinder implem
     }
 
     /**
-     * {@inheritDoc}
-     */
-    public <T> ConverterBuilder<T> convert(Class<T> type) {
-        return this.wrappedRulesBinder.convert(type);
-    }
-
-    /**
      * 
      * @param bindingClass
      * @return

Modified: 
commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/internal/DigesterImpl.java
URL: 
http://svn.apache.org/viewvc/commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/internal/DigesterImpl.java?rev=1072614&r1=1072613&r2=1072614&view=diff
==============================================================================
--- 
commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/internal/DigesterImpl.java
 (original)
+++ 
commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/internal/DigesterImpl.java
 Sun Feb 20 15:40:57 2011
@@ -17,7 +17,11 @@
  */
 package org.apache.commons.digester3.internal;
 
-import static org.apache.commons.digester3.utils.InputSources.*;
+import static 
org.apache.commons.digester3.utils.InputSources.createInputSourceFromFile;
+import static 
org.apache.commons.digester3.utils.InputSources.createInputSourceFromInputStream;
+import static 
org.apache.commons.digester3.utils.InputSources.createInputSourceFromReader;
+import static 
org.apache.commons.digester3.utils.InputSources.createInputSourceFromURL;
+import static 
org.apache.commons.digester3.utils.InputSources.createInputSourceFromUri;
 
 import java.io.File;
 import java.io.IOException;
@@ -37,7 +41,6 @@ import org.apache.commons.digester3.Dige
 import org.apache.commons.digester3.Rule;
 import org.apache.commons.digester3.spi.Rules;
 import org.apache.commons.digester3.spi.Substitutor;
-import org.apache.commons.digester3.spi.TypeConverter;
 import org.apache.commons.logging.Log;
 import org.apache.commons.logging.LogFactory;
 import org.xml.sax.Attributes;
@@ -1167,12 +1170,4 @@ public final class DigesterImpl implemen
         return createSAXException(message, null);
     }
 
-    /**
-     * {@inheritDoc}
-     */
-    public <T> TypeConverter<T> lookupConverter(final Class<T> type) {
-        // TODO empty implementation, will be changed for final version
-        return null;
-    }
-
 }

Modified: 
commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/internal/rulesbinder/RulesBinderImpl.java
URL: 
http://svn.apache.org/viewvc/commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/internal/rulesbinder/RulesBinderImpl.java?rev=1072614&r1=1072613&r2=1072614&view=diff
==============================================================================
--- 
commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/internal/rulesbinder/RulesBinderImpl.java
 (original)
+++ 
commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/internal/rulesbinder/RulesBinderImpl.java
 Sun Feb 20 15:40:57 2011
@@ -28,10 +28,8 @@ import org.apache.commons.digester3.Dige
 import org.apache.commons.digester3.Rule;
 import org.apache.commons.digester3.RulesBinder;
 import org.apache.commons.digester3.RulesModule;
-import org.apache.commons.digester3.rulesbinder.ConverterBuilder;
 import org.apache.commons.digester3.rulesbinder.LinkedRuleBuilder;
 import org.apache.commons.digester3.spi.Rules;
-import org.apache.commons.digester3.spi.TypeConverter;
 
 /**
  * The Digester EDSL implementation.
@@ -139,28 +137,6 @@ public final class RulesBinderImpl imple
     }
 
     /**
-     * {@inheritDoc}
-     */
-    public <T> ConverterBuilder<T> convert(final Class<T> type) {
-        if (type == null) {
-            this.addError(new IllegalArgumentException("NULL type is not 
allowed to be converted"));
-        }
-        return new ConverterBuilder<T>() {
-
-            public void withConverter(TypeConverter<T> typeConverter) {
-                if (typeConverter == null) {
-                    addError(new IllegalArgumentException(
-                            String.format("NULL TypeConverter is not allowed 
for converting '%s' type",
-                                    type.getName())));
-                }
-
-                // TODO register the type converter!!!
-            }
-
-        };
-    }
-
-    /**
      * Invokes the bound providers, then create the rule and associate it to 
the related pattern,
      * storing them in the proper {@link Rules} implementation data structure.
      *

Modified: 
commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/xmlrules/metaparser/PrefixedRulesBinder.java
URL: 
http://svn.apache.org/viewvc/commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/xmlrules/metaparser/PrefixedRulesBinder.java?rev=1072614&r1=1072613&r2=1072614&view=diff
==============================================================================
--- 
commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/xmlrules/metaparser/PrefixedRulesBinder.java
 (original)
+++ 
commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/xmlrules/metaparser/PrefixedRulesBinder.java
 Sun Feb 20 15:40:57 2011
@@ -19,7 +19,6 @@ package org.apache.commons.digester3.xml
 
 import org.apache.commons.digester3.RulesBinder;
 import org.apache.commons.digester3.RulesModule;
-import org.apache.commons.digester3.rulesbinder.ConverterBuilder;
 import org.apache.commons.digester3.rulesbinder.LinkedRuleBuilder;
 
 final class PrefixedRulesBinder implements RulesBinder {
@@ -71,11 +70,4 @@ final class PrefixedRulesBinder implemen
         return this.wrappedRulesBinder.forPattern(pattern);
     }
 
-    /**
-     * {@inheritDoc}
-     */
-    public <T> ConverterBuilder<T> convert(Class<T> type) {
-        return this.wrappedRulesBinder.convert(type);
-    }
-
 }

Modified: 
commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/xmlrules/metaparser/WithMemoryRulesBinder.java
URL: 
http://svn.apache.org/viewvc/commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/xmlrules/metaparser/WithMemoryRulesBinder.java?rev=1072614&r1=1072613&r2=1072614&view=diff
==============================================================================
--- 
commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/xmlrules/metaparser/WithMemoryRulesBinder.java
 (original)
+++ 
commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/xmlrules/metaparser/WithMemoryRulesBinder.java
 Sun Feb 20 15:40:57 2011
@@ -22,7 +22,6 @@ import java.util.Set;
 
 import org.apache.commons.digester3.RulesBinder;
 import org.apache.commons.digester3.RulesModule;
-import org.apache.commons.digester3.rulesbinder.ConverterBuilder;
 import org.apache.commons.digester3.rulesbinder.LinkedRuleBuilder;
 
 /**
@@ -85,13 +84,6 @@ class WithMemoryRulesBinder implements R
     }
 
     /**
-     * {@inheritDoc}
-     */
-    public <T> ConverterBuilder<T> convert(Class<T> type) {
-        return this.wrappedRulesBinder.convert(type);
-    }
-
-    /**
      * 
      *
      * @return


Reply via email to