You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Philip Martin <ph...@codematters.co.uk> on 2003/02/04 19:38:58 UTC
Re: svn commit: rev 4721 - in trunk/subversion: clients/cmdline svnversion
rooneg@tigris.org writes:
> Author: rooneg
> Date: 2003-02-03 19:25:46 -0600 (Mon, 03 Feb 2003)
> New Revision: 4721
>
> Modified:
> trunk/subversion/clients/cmdline/main.c
> trunk/subversion/svnversion/main.c
> Log:
> * subversion/clients/cmdline/main.c
> (main): replace memset of opt_state and ctx with initialization to 0 at
> declaration time.
>
> * subversion/svnversion/main.c
> (main): replace memset of ctx with initialization to 0 at declaration time.
>
>
> Modified: trunk/subversion/clients/cmdline/main.c
> ==============================================================================
> --- trunk/subversion/clients/cmdline/main.c (original)
> +++ trunk/subversion/clients/cmdline/main.c Mon Feb 3 19:25:51 2003
> @@ -511,8 +511,8 @@
> apr_pool_t *pool;
> int opt_id, err2;
> apr_getopt_t *os;
> - svn_cl__opt_state_t opt_state;
> - svn_client_ctx_t ctx;
> + svn_cl__opt_state_t opt_state = { 0 };
The first member of svn_cl__opt_state_t is a structure, so 0 is not a
valid initialiser
../svn/subversion/clients/cmdline/main.c: In function `main':
../svn/subversion/clients/cmdline/main.c:514: warning: missing braces around initializer
../svn/subversion/clients/cmdline/main.c:514: warning: (near initialization for `opt_state.start_revision')
I suggest either
svn_cl__opt_state_t opt_state = { };
or
svn_cl__opt_state_t opt_state = { { 0 } };
--
Philip Martin
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Re: svn commit: rev 4721 - in trunk/subversion: clients/cmdline svnversion
Posted by Garrett Rooney <ro...@electricjellyfish.net>.
Philip Martin wrote:
>The first member of svn_cl__opt_state_t is a structure, so 0 is not a
>valid initialiser
>
>../svn/subversion/clients/cmdline/main.c: In function `main':
>../svn/subversion/clients/cmdline/main.c:514: warning: missing braces around initializer
>../svn/subversion/clients/cmdline/main.c:514: warning: (near initialization for `opt_state.start_revision')
>
>I suggest either
>
> svn_cl__opt_state_t opt_state = { };
>
>or
>
> svn_cl__opt_state_t opt_state = { { 0 } };
>
damn. it worked fine on my os x box... either solution is fine with me
as long as it results in the structure being zero'd out... feel free to
commit now if you like, as i won't be able to take a look till later
tonight.
-garrett
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org