svn commit: r453208 - /maven/scm/trunk/maven-scm-providers/maven-scm-providers-svn/maven-scm-provider-svn-commons/src/main/java/org/apache/maven/scm/provider/svn/AbstractSvnScmProvider.java

[kenney]

Author: kenney
Date: Thu Oct  5 05:44:05 2006
New Revision: 453208

URL: http://svn.apache.org/viewvc?view=rev&rev=453208
Log:
Fix & Detect NPE

Modified:
    
maven/scm/trunk/maven-scm-providers/maven-scm-providers-svn/maven-scm-provider-svn-commons/src/main/java/org/apache/maven/scm/provider/svn/AbstractSvnScmProvider.java

Modified: 
maven/scm/trunk/maven-scm-providers/maven-scm-providers-svn/maven-scm-provider-svn-commons/src/main/java/org/apache/maven/scm/provider/svn/AbstractSvnScmProvider.java
URL: 
http://svn.apache.org/viewvc/maven/scm/trunk/maven-scm-providers/maven-scm-providers-svn/maven-scm-provider-svn-commons/src/main/java/org/apache/maven/scm/provider/svn/AbstractSvnScmProvider.java?view=diff&rev=453208&r1=453207&r2=453208
==============================================================================
--- 
maven/scm/trunk/maven-scm-providers/maven-scm-providers-svn/maven-scm-provider-svn-commons/src/main/java/org/apache/maven/scm/provider/svn/AbstractSvnScmProvider.java
 (original)
+++ 
maven/scm/trunk/maven-scm-providers/maven-scm-providers-svn/maven-scm-provider-svn-commons/src/main/java/org/apache/maven/scm/provider/svn/AbstractSvnScmProvider.java
 Thu Oct  5 05:44:05 2006
@@ -48,7 +48,7 @@
 
 /**
  * SCM Provider for Subversion
- * 
+ *
  * @author <a href="mailto:[EMAIL PROTECTED]">Emmanuel Venisse</a>
  * @version $Id$
  */
@@ -94,7 +94,12 @@
     public ScmProviderRepository makeProviderScmRepository( File path )
         throws ScmRepositoryException, UnknownRepositoryStructure
     {
-        if ( path == null || !path.isDirectory() )
+        if ( path == null )
+        {
+            throw new NullPointerException( "Path argument is null" );
+        }
+
+        if ( !path.isDirectory() )
         {
             throw new ScmRepositoryException( path.getAbsolutePath() + " isn't 
a valid directory." );
         }