maven-scm-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From d...@multitask.com.au
Subject Re: cvs commit: maven-scm/generic-scm-providers/clearcase/src/java/org/apache/maven/genericscm/provider/clearcase/command ClearcaseCommandWrapper.java
Date Sat, 06 Dec 2003 14:15:05 GMT
evenisse@apache.org wrote on 07/12/2003 12:56:43 AM:

> evenisse    2003/12/06 05:56:43
> 
>   Added:       generic-scm-
> 
providers/clearcase/src/java/org/apache/maven/genericscm/provider/clearcase/command
>                         ClearcaseCommandWrapper.java
>   Log:
>   Add command wrapper
> 
>       /* (non-Javadoc)
>        * @see org.apache.maven.genericscm.command.
> 
CommandWrapper#setRepository(org.apache.maven.genericscm.repository.Repository)
>        */
>       public void setRepository(Repository repository) throws 
ScmException
>       {
>           if (repository instanceof ClearcaseRepository)
>           {
>               this.repository = (ClearcaseRepository)repository;
> 
>               for (Enumeration e = getCommandsTable().elements(); e.
> hasMoreElements(); )
>               {
>                   Command command = (Command)e.nextElement();
>                   command.setRepository(repository);
>               }
>           }
>           throw new ScmException("repository must be an instance of 
> ClearcaseRepository");
>       }

Shouldn't there be an 'else' before that throw?
--
dIon Gillard, Multitask Consulting
Blog:      http://blogs.codehaus.org/people/dion/




Mime
View raw message