ant-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From dona...@apache.org
Subject cvs commit: jakarta-ant/proposal/myrmidon/src/java/org/apache/myrmidon/components/type DefaultTypeManager.java
Date Fri, 08 Jun 2001 09:09:46 GMT
donaldp     01/06/08 02:09:46

  Modified:    proposal/myrmidon/src/java/org/apache/myrmidon/components/type
                        DefaultTypeManager.java
  Log:
  Removed decoration of role with "selector" as it was legacy to old implementation.
  
  Revision  Changes    Path
  1.6       +3 -7      jakarta-ant/proposal/myrmidon/src/java/org/apache/myrmidon/components/type/DefaultTypeManager.java
  
  Index: DefaultTypeManager.java
  ===================================================================
  RCS file: /home/cvs/jakarta-ant/proposal/myrmidon/src/java/org/apache/myrmidon/components/type/DefaultTypeManager.java,v
  retrieving revision 1.5
  retrieving revision 1.6
  diff -u -r1.5 -r1.6
  --- DefaultTypeManager.java	2001/06/03 06:10:05	1.5
  +++ DefaultTypeManager.java	2001/06/08 09:09:45	1.6
  @@ -38,14 +38,14 @@
                                 final TypeFactory factory ) 
           throws TypeException
       {
  -        final MultiSourceTypeFactory msFactory = createFactory( role + "Selector" );
  +        final MultiSourceTypeFactory msFactory = createFactory( role );
           msFactory.register( shorthandName, factory );
       }
   
       public TypeFactory getFactory( final String role )
           throws TypeException
       {
  -        return createFactory( role + "Selector" );
  +        return createFactory( role );
       }
   
       protected final MultiSourceTypeFactory lookupFactory( final String role )
  @@ -79,14 +79,10 @@
           ///If we haven't goa factory try to create a new one
           if( null == factory )
           {
  -            //Precondition that role.endsWith( "Factory" )
  -            final int length = role.length() - 8;
  -            final String workInterface = role.substring( 0, length );
  -
               try
               {
                   //TODO: Should we use ContextClassLoader here ??? Or perhaps try that on
failure??
  -                final Class clazz = Class.forName( workInterface );
  +                final Class clazz = Class.forName( role );
                   factory = new MultiSourceTypeFactory( clazz );
               }
               catch( final Exception e )
  
  
  

Mime
View raw message