You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by br...@apache.org on 2013/07/11 06:32:52 UTC

svn commit: r1502115 - /subversion/trunk/subversion/bindings/javahl/native/RemoteSession.cpp

Author: brane
Date: Thu Jul 11 04:32:52 2013
New Revision: 1502115

URL: http://svn.apache.org/r1502115
Log:
Fix a bug in the JavaHL RA API where the default config directory was not used
when a remote session was created through RemoteFactory.

* subversion/bindings/javahl/native/RemoteSession.cpp
  (RemoteSession::open): Use NULL for the config directory path, not an empty
   string, if the Java string parameter was null.

Modified:
    subversion/trunk/subversion/bindings/javahl/native/RemoteSession.cpp

Modified: subversion/trunk/subversion/bindings/javahl/native/RemoteSession.cpp
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/bindings/javahl/native/RemoteSession.cpp?rev=1502115&r1=1502114&r2=1502115&view=diff
==============================================================================
--- subversion/trunk/subversion/bindings/javahl/native/RemoteSession.cpp (original)
+++ subversion/trunk/subversion/bindings/javahl/native/RemoteSession.cpp Thu Jul 11 04:32:52 2013
@@ -110,8 +110,8 @@ RemoteSession::open(jint jretryAttempts,
 
   jobject jremoteSession = open(
       jretryAttempts, url.c_str(), uuid,
-      configDirectory.c_str(), jconfigHandler,
-      usernameStr, passwordStr, prompter, jprogress);
+      (jconfigDirectory ? configDirectory.c_str() : NULL),
+      jconfigHandler, usernameStr, passwordStr, prompter, jprogress);
   if (JNIUtil::isExceptionThrown() || !jremoteSession)
     {
       delete prompter;