On Fri, Mar 12, 2010 at 08:15, neels <[email protected]> wrote: > On 12 March 2010 04:11, Neels J Hofmeyr <[email protected]> wrote: >> Greg Stein wrote: >>>> + else >>>> + if (status == svn_wc__db_status_base_deleted) >>> >>> Woah. This formatting is incorrect. We always do "else if (...". The > > So I've grep'd around for else(\n)if occurences. > Notably I'm the only one that lined up the 'if' with the 'else'. All > other occurences indent the 'if'. > > hyrum, gstein, would you be fine with me adopting the indented-if > variant? It pleases my eyes *slightly* more. Feel free to say "nah, > come on, just use 'else if' and done".
You found 10 occurrences of else\n if. I found 1239 occurrences of else if. Guess my answer :-) Cheers, -g
