Re: Results of: [VOTE] New space-before-parens style

Lares Moreau wrote:
> On Sun, 2006-02-12 at 20:56 -0600, kfogel@collab.net wrote:
>> Anyone got any better ideas, before I go further down that road?
>>
>> -Karl
>
> I'm thinking more along the lines of sed.
>
> $ svn info | grep "^URL"
> URL: http://svn.collab.net/repos/svn/trunk/subversion/svn
> $ sed --version
> GNU sed version 4.1.4
> $ find -type f -iname "*.[ch]" | xargs -n1 \
> sed 's:\([a-z0-9_]\+\) ([^\(c?)\)]:\1(:g' -i
> $ svn diff |head -n50
> Index: merge-cmd.c
> ===================================================================
> --- merge-cmd.c (revision 18446)
> +++ merge-cmd.c (working copy)
[...]
> - if (opt_state->start_revision.kind != svn_opt_revision_unspecified)
> + if(pt_state->start_revision.kind != svn_opt_revision_unspecified)

I did not think we were changing if/while/etc spacing. That is, if (...)
and not if(...) since "if" is not a function. (Same for while/switch/etc)

-- 
Michael Sinz                     Technology and Engineering Director/Consultant
"Starting Startups"                                mailto:michael.sinz@sinz.org
My place on the web                            http://www.sinz.org/Michael.Sinz
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org