Hez,

I also spotted this, and I entirely agree with you.

Andrew

On Thu, Oct 22, 2009 at 04:07:30PM -0500, Hezekiah M. Carty wrote:
> Alan,
> 
> While reading through the reformatted C in the OCaml bindings portion
> of your recent uncrustify commits, I noticed a small inconsistency.
> When a function call ends with "));", not extra space is added to make
> this ") );".  This seems to be consistent throughout the rest of the
> reformatted C source code.
> 
> I see that the relevant setting in uncrustify.cfg (sp_paren_paren) is
> set to remove these spaces.  Is there a reason for this?  While
> reading through the changes I found the new formatting hindering my
> visual tracking through the code.
> 
> If there are no objections I would like this option changed to include
> spaces between )) and ((.  My apologies for not bringing this up
> sooner - I had not noticed this issue in the reformatted C before now.
> 
> Hez
> 
> -- 
> Hezekiah M. Carty
> Graduate Research Assistant
> University of Maryland
> Department of Atmospheric and Oceanic Science
> 
> ------------------------------------------------------------------------------
> Come build with us! The BlackBerry(R) Developer Conference in SF, CA
> is the only developer event you need to attend this year. Jumpstart your
> developing skills, take BlackBerry mobile applications to market and stay 
> ahead of the curve. Join us from November 9 - 12, 2009. Register now!
> http://p.sf.net/sfu/devconference
> _______________________________________________
> Plplot-devel mailing list
> Plplot-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/plplot-devel
> 

------------------------------------------------------------------------------
Come build with us! The BlackBerry(R) Developer Conference in SF, CA
is the only developer event you need to attend this year. Jumpstart your
developing skills, take BlackBerry mobile applications to market and stay 
ahead of the curve. Join us from November 9 - 12, 2009. Register now!
http://p.sf.net/sfu/devconference
_______________________________________________
Plplot-devel mailing list
Plplot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/plplot-devel

Reply via email to