| 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-0600 | Date Closed: | 2009-12-04 11:09:13.000-0600 |
| Priority: | Trivial | Regression? | No |
| Status: | Closed/Complete | Components: | General |
| Versions: | Frequency of Occurrence | ||
| Related Issues: | |||
| 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. | ||