On Sun, May 19, 2013 at 10:15:55AM -0000, g...@apache.org wrote: > Author: gbg > Date: Sun May 19 10:15:55 2013 > New Revision: 1484260 > > URL: http://svn.apache.org/r1484260 > Log: > Seperate variable declaration from assigment. > > * subversion/libsvn_client/diff.c > (set_up_diff_cmd_and_options): Seperate variable declaration from assigment. > > Modified: > > subversion/branches/invoke-diff-cmd-feature/subversion/libsvn_client/diff.c > > Modified: > subversion/branches/invoke-diff-cmd-feature/subversion/libsvn_client/diff.c > URL: > http://svn.apache.org/viewvc/subversion/branches/invoke-diff-cmd-feature/subversion/libsvn_client/diff.c?rev=1484260&r1=1484259&r2=1484260&view=diff > ============================================================================== > --- > subversion/branches/invoke-diff-cmd-feature/subversion/libsvn_client/diff.c > (original) > +++ > subversion/branches/invoke-diff-cmd-feature/subversion/libsvn_client/diff.c > Sun May 19 10:15:55 2013 > @@ -2465,7 +2465,9 @@ set_up_diff_cmd_and_options(struct diff_ > /* old style diff_cmd has precedence in config file */ > if (config) > { > - svn_config_t *cfg = svn_hash_gets(config, SVN_CONFIG_CATEGORY_CONFIG); > + svn_config_t *cfg; > + > + cfg = svn_hash_gets(config, SVN_CONFIG_CATEGORY_CONFIG);
Why? This doesn't seem to serve any useful purpose (in fact, I think it makes the code harder to read).
