[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [patch] c99-isms

From: Matthew Woehlke <mw_triad_at_users.sourceforge.net>
Date: Thu, 26 Mar 2009 12:15:23 -0500

Branko Cibej wrote:
> Also ... you actually *can* init almost all of argv here. :)
>
> const char *argv[] = { "opt-test", NULL, NULL };
>
> and later:
>
> argv[1] = input;

Yes, it's a stylistic decision. Filling a list that is effectively a
bunch of pointers plus a NULL to mark the end, it feels a bit strange to
me to do some constant initialization and then "inject" things, versus
an explicit assignment to terminate it. But as long as it compiles, I
really don't care what style you prefer :-). I just picked one and went
with it.

-- 
Matthew
Please do not quote my e-mail address unobfuscated in message bodies.
-- 
"We're all mad here. I'm mad. You're mad... You must be, or you wouldn't 
have come here." -- The Cheshire Cat (from Lewis Carroll's Alice in 
Wonderland)
------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=1432547
Received on 2009-03-26 18:34:41 CET

This is an archived mail posted to the Subversion Dev mailing list.