Martin Furter wrote:
> I thought i could add 2 commandline switches, --auto-props and
> --no-auto-props, so enabling and disabling is both possible, and when they
> are found on the cmdline the value of the config is overwritten with
> svn_config_set().
> This way i don't need to change any API or struct. Is that OK ?
That sounds good to me.
> +/* Method which automatically adds properties.
> +
> + @param path a filename
> + @param ctx the client context
> + @param pool apr memory pool
> + @param adm_access whatever ???, used when adding
> + @param editor editor, used when importing
> + @param file_baton file baton for editor, used when importing
> + @param mimetype pointer for returning mimetype, used when importing
> + @param executable pointer for returning executable, used when importing
> + @return an svn_error_t or SVN_NO_ERROR when everything went OK
> +*/
> +svn_error_t *
> +svn_client__add_auto_props (const char *path,
> + svn_client_ctx_t *ctx,
> + apr_pool_t *pool,
> + svn_wc_adm_access_t *adm_access,
> + const svn_delta_editor_t *editor,
> + void *file_baton,
> + const char **mimetype,
> + int *executable);
> +
I haven't had a chance to look closely at the patch, but in order to get
doxygen to pick up your @param tags you'll have to start the comment
with /** like the rest of the doxygen comments in svn_client.h. Also,
our documentation comments tend to be a bit more conversational than
that, see the rest of the file for examples.
-garrett
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sun Aug 17 23:39:07 2003