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;