On Mon, Jun 7, 2010 at 16:35,  <rhuij...@apache.org> wrote:
>...
> Modified: subversion/trunk/subversion/bindings/javahl/native/CreateJ.cpp
> URL: 
> http://svn.apache.org/viewvc/subversion/trunk/subversion/bindings/javahl/native/CreateJ.cpp?rev=952421&r1=952420&r2=952421&view=diff
> ==============================================================================
> --- subversion/trunk/subversion/bindings/javahl/native/CreateJ.cpp (original)
> +++ subversion/trunk/subversion/bindings/javahl/native/CreateJ.cpp Mon Jun  7 
> 20:35:54 2010
> @@ -567,9 +567,18 @@ CreateJ::Status(svn_wc_context_t *wc_ctx
>       if (JNIUtil::isJavaExceptionThrown())
>         POP_AND_RETURN_NULL;
>
> -      const svn_wc_entry_t *entry = status->entry;
> +      const svn_wc_entry_t *entry = NULL;
> +
> +      if (status->versioned)
> +        /* ### This doesn't set enty when _get_entry returns
> +           ### SVN_ERR_NODE_UNEXPECTED_KIND! */
> +        SVN_JNI_ERR(svn_wc__get_entry_versioned(&entry, wc_ctx, 
> local_abspath,
> +                                                svn_node_unknown, FALSE, 
> FALSE,
> +                                                pool, pool));

Hunh? entry IS set when that error is returned. (and a typo)

Given the above parameters, you'll get UNEXPECTED_KIND if the subdir
is missing/obstructed. The entry will contain the parent stub (yet you
asked for the "real" entry).

>...

Cheers,
-g

Reply via email to