Todo List

Global svn_client_cat2 (svn_stream_t *out, const char *path_or_url, const svn_opt_revision_t *peg_revision, const svn_opt_revision_t *revision, svn_client_ctx_t *ctx, apr_pool_t *pool)

Add an expansion/translation flag?

Global svn_client_version (void)

Multiple Targets

• Up for debate: an update on multiple targets is *not* atomic.

Global svn_fs_copied_from (svn_revnum_t *rev_p, const char **path_p, svn_fs_root_t *root, const char *path, apr_pool_t *pool)

See how people like this, it can always be hidden behind the curtain if necessary.

Global svn_fs_file_contents (svn_stream_t **contents, svn_fs_root_t *root, const char *path, apr_pool_t *pool)

kff: I am worried about lifetime issues with this pool vs the trail created farther down the call stack. Trace this function to investigate...

Global svn_opt_print_generic_help2 (const char *header, const svn_opt_subcommand_desc2_t *cmd_table, const apr_getopt_option_t *opt_table, const char *footer, apr_pool_t *pool, FILE *stream)

Why is stream a stdio file instead of an svn stream?

Global svn_opt_subcommand_help3 (const char *subcommand, const svn_opt_subcommand_desc2_t *table, const apr_getopt_option_t *options_table, const int *global_options, apr_pool_t *pool)

Why does this only print to stdout, whereas svn_opt_print_generic_help() gives us a choice?

Global svn_path_is_child (const char *path1, const char *path2, apr_pool_t *pool)

the ".." restriction is unfortunate, and would ideally be lifted by making the implementation smarter. But this is not trivial: if the path is "../foo", how do you know whether or not the current directory is named "foo" in its parent?

Global svn_wc_add3 (const char *path, svn_wc_adm_access_t *parent_access, svn_depth_t depth, const char *copyfrom_url, svn_revnum_t copyfrom_rev, svn_cancel_func_t cancel_func, void *cancel_baton, svn_wc_notify_func2_t notify_func, void *notify_baton, apr_pool_t *pool)

Remove old wcprops too, see the '###' below.

Global svn_wc_conflict_version_t

Consider making some or all of the info mandatory, to reduce complexity.