<div dir="ltr">Ping! Please let me know if there are any changes you would like me to make, or if this is ready to commit.<div><br></div><div>Thanks,</div><div>Aaron</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">
On Thu, Apr 3, 2014 at 9:25 AM, Aaron Wishnick <span dir="ltr"><<a href="mailto:aaron.s.wishnick@gmail.com" target="_blank">aaron.s.wishnick@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr"><span style="font-family:arial,sans-serif;font-size:13px">This patch adds an option to control when spaces are added inside of parentheses. My employer's style guide mandates that spaces go inside the outermost pair of parentheses, but not the rest. For example:</span><div style="font-family:arial,sans-serif;font-size:13px">

<br></div><div style="font-family:arial,sans-serif;font-size:13px">if( someFunction(a, b, c) ) {</div><div style="font-family:arial,sans-serif;font-size:13px">  doThing( f(x), f(g(x)) );</div><div style="font-family:arial,sans-serif;font-size:13px">

}</div><div style="font-family:arial,sans-serif;font-size:13px"><br></div><div style="font-family:arial,sans-serif;font-size:13px">My attached patch implements this feature with a new option, SpacesInParenthesesStyle, which can either be "Always" (the previous behavior, and the default), or "Outermost", the new behavior used by my organization.</div>

<div style="font-family:arial,sans-serif;font-size:13px"><br></div><div style="font-family:arial,sans-serif;font-size:13px">Does this seem like a reasonable strategy? The new option defaults to the previous behavior. I see that a different approach was taken with SpaceBeforeParensOptions. I went with this approach, because the option applies to spaces inserted inside of parentheses due to SpacesInParentheses, SpaceInEmptyParentheses, SpacesInCStyleCastParentheses, etc.</div>

<div style="font-family:arial,sans-serif;font-size:13px"><br></div><div style="font-family:arial,sans-serif;font-size:13px">Thanks,<br>Aaron</div><div style="font-family:arial,sans-serif;font-size:13px"><br></div><div style="font-family:arial,sans-serif;font-size:13px">

N.B. I originally sent this to the cfe-dev list by mistake, sorry for the noise!</div></div>
</blockquote></div><br></div>