[Home]

Summary:ASTERISK-05009: [patch] [post 1.2] Switch indentation
Reporter:Jose Pablo Fernandez (pupeno)Labels:
Date Opened:2005-09-06 14:33:23Date Closed:2011-06-07 14:11:56
Priority:TrivialRegression?No
Status:Closed/CompleteComponents:Channels/chan_agent
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) channel.c-switch-indentation.patch
Description:Switch indentation on channel.c did not respect the guideline http://voip-info.org/wiki-Asterisk+Developer+Guidelines
This patch solves that.
Comments:By: Clod Patry (junky) 2005-09-06 19:43:13

I've no idea who wrote that wiki, but that's false.
If you take a look on doc/CODING-GUIDELINES
[...]
Case statements:
switch (foo) {
case BAR:
       blah();
       break;
case OTHER:
       other();
       break;
}
[...]

So i see no need for that patch.


That would be a great idea to change that wiki.

Thanks.