On Thu, Mar 18, 2010 at 07:34, <[email protected]> wrote: > Author: julianfoad > Date: Thu Mar 18 11:34:45 2010 > New Revision: 924722 > > URL: http://svn.apache.org/viewvc?rev=924722&view=rev > Log: > Clean up two revved WC API function signatures: move the wc_ctx parameter > before the first of the main input parameters, to keep those together and > to minimize the difference from the previous version. Do some doc string > clean-ups too. > > * subversion/include/svn_wc.h > (svn_wc_get_status_editor5): Fix a parameter name typo. Note doc string > out-of-dateness. > (svn_wc_crawl_revisions5): Note doc string out-of-dateness. > (svn_wc_get_update_editor4): Swap revision and wc_ctx parameters. > (svn_wc_get_switch_editor4): Swap revision and wc_ctx parameters. Refer to > svn_wc_get_update_editor4's doc string for most parameters to make it > easy for the reader to see that there are no differences.
Why swap these parameters? REVISION (or TARGET_REVISION) is an OUT parameter. We always place OUT parameters first, and then the WC_CTX, then the other params. >... Cheers, -g
