You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by pe...@apache.org on 2010/10/04 22:22:02 UTC
svn commit: r1004397 - /subversion/trunk/subversion/svn/main.c
Author: peters
Date: Mon Oct 4 20:22:02 2010
New Revision: 1004397
URL: http://svn.apache.org/viewvc?rev=1004397&view=rev
Log:
Followup r1004314: Tighten scopes in a few more variables.
* subversion/svn/main.c
(main): Tighten scopes for a few variables.
Modified:
subversion/trunk/subversion/svn/main.c
Modified: subversion/trunk/subversion/svn/main.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/svn/main.c?rev=1004397&r1=1004396&r2=1004397&view=diff
==============================================================================
--- subversion/trunk/subversion/svn/main.c (original)
+++ subversion/trunk/subversion/svn/main.c Mon Oct 4 20:22:02 2010
@@ -1278,8 +1278,6 @@ main(int argc, const char *argv[])
int i;
const svn_opt_subcommand_desc2_t *subcommand = NULL;
const char *dash_m_arg = NULL, *dash_F_arg = NULL;
- const char *path_utf8;
- apr_status_t apr_err;
svn_cl__cmd_baton_t command_baton;
svn_auth_baton_t *ab;
svn_config_t *cfg_config;
@@ -1357,7 +1355,7 @@ main(int argc, const char *argv[])
const char *utf8_opt_arg;
/* Parse the next option. */
- apr_err = apr_getopt_long(os, svn_cl__options, &opt_id, &opt_arg);
+ apr_status_t apr_err = apr_getopt_long(os, svn_cl__options, &opt_id, &opt_arg);
if (APR_STATUS_IS_EOF(apr_err))
break;
else if (apr_err)
@@ -1398,9 +1396,6 @@ main(int argc, const char *argv[])
break;
case 'c':
{
- char *end;
- svn_revnum_t changeno;
- svn_opt_revision_range_t *range;
apr_array_header_t *change_revs =
svn_cstring_split(opt_arg, ", \n\r\t\v", TRUE, pool);
@@ -1414,6 +1409,9 @@ main(int argc, const char *argv[])
for (i = 0; i < change_revs->nelts; i++)
{
+ char *end;
+ svn_revnum_t changeno;
+ svn_opt_revision_range_t *range;
const char *change_str =
APR_ARRAY_IDX(change_revs, i, const char *);
@@ -1665,10 +1663,13 @@ main(int argc, const char *argv[])
opt_state.new_target = apr_pstrdup(pool, opt_arg);
break;
case opt_config_dir:
- err = svn_utf_cstring_to_utf8(&path_utf8, opt_arg, pool);
- if (err)
- return svn_cmdline_handle_exit_error(err, pool, "svn: ");
- opt_state.config_dir = svn_dirent_internal_style(path_utf8, pool);
+ {
+ const char *path_utf8;
+ err = svn_utf_cstring_to_utf8(&path_utf8, opt_arg, pool);
+ if (err)
+ return svn_cmdline_handle_exit_error(err, pool, "svn: ");
+ opt_state.config_dir = svn_dirent_internal_style(path_utf8, pool);
+ }
break;
case opt_config_options:
if (!opt_state.config_options)