Hyrum or anyone,
svn_client_status_func_t is declared as
/** A callback for reporting a @a status about @a local_abspath.
... */
typedef svn_error_t *(*svn_client_status_func_t)(
void *baton,
const char *path,
const svn_client_status_t *status,
apr_pool_t *scratch_pool);
but its "path" parameter is not called "local_abspath" and is not
guaranteed to be an abspath, which is evident from the implementation.
I am satisfied that all but one of the current callbacks is happy with a
non-absolute path. The only one I can't validate is the JavaHL binding:
SVNClient::status() in SVNClient.cpp
where, according to the parameter names, it expects an abspath.
Can someone better able to follow the Java code check this?
Then I think we just need to adjust the doc string to say something like
"path, which may be absolute or relative", and rename the JavaHL
parameters to "path".
- Julian
