Summary:ASTERISK-15086: [patch] if, for, while, switch statements all missing space, - Coding guidelines
Reporter:Alec Davis (alecdavis)Labels:
Date Opened:2009-11-06 01:37:58.000-0600Date Closed:2009-12-04 11:09:13.000-0600
Versions:Frequency of
Environment:Attachments:( 0) asterisk-if-for-when-switch.diff.txt
Description:Global change for all *.c files under the asterisk where whole word matches any of the following.
'if(', 'for(', 'while(' or 'switch('

no functional changes.  
Comments:By: Leif Madsen (lmadsen) 2009-11-06 09:19:58.000-0600

Duplicate of issue DAHLIN-160

By: Alec Davis (alecdavis) 2009-11-06 13:48:35.000-0600

Is not duplicate. 16191 is for DAHDI.
This is for asterisk.

By: Russell Bryant (russell) 2009-11-06 14:27:03.000-0600

I'm really hesitant to commit stuff like this because it's hard to say that it's worth the pain it causes by breaking existing patches and causing conflicts when merging patches between branches.  I'm certainly all for clean up when you're changing an area of code anyway, but I'm not so fond of mass changes like this.

By: Russell Bryant (russell) 2009-12-04 11:09:13.000-0600

As I previously indicated, I'd rather not merge this one.  Thanks, though.