update guidelines to follow their own rules for whitespace (bug #4486)
authorKevin P. Fleming <kpfleming@digium.com>
Tue, 7 Jun 2005 21:28:56 +0000 (21:28 +0000)
committerKevin P. Fleming <kpfleming@digium.com>
Tue, 7 Jun 2005 21:28:56 +0000 (21:28 +0000)
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@5880 65c4cc65-6c06-0410-ace0-fbb531ad65f3

doc/CODING-GUIDELINES

index cc352f7..cf97f68 100755 (executable)
@@ -90,10 +90,10 @@ for (x=0;x<5;x++) {
 Don't build code like this:
 
 if (foo) {
-   .... 50 lines of code ...
+       /* .... 50 lines of code ... */
 } else {
-  result = 0;
-  return;
+       result = 0;
+       return;
 }
 
 Instead, try to minimize the number of lines of code that need to be
@@ -101,8 +101,8 @@ indented, by only indenting the shortest case of the 'if'
 statement, like so:
 
 if !(foo) {
-  result = 0;
-  return;
+       result = 0;
+       return;
 }
 
 .... 50 lines of code ....
@@ -125,13 +125,14 @@ options that they are in fact intended to be global.
 
 When making applications, always ast_strdupa(data) to a local pointer if
 you intend to parse it.
- if(data)
-  mydata = ast_strdupa(data);
+
+       if (data)
+               mydata = ast_strdupa(data);
 
 Always derefrence or localize pointers to things that are not yours like
 channel members in a channel that is not associated with the current 
 thread and for which you do not have a lock.
- channame = ast_strdupa(otherchan->name);
+       channame = ast_strdupa(otherchan->name);
 
 If you do the same or a similar operation more than 1 time, make it a
 function or macro.
@@ -200,7 +201,7 @@ struct foo *tmp;
 
 tmp = malloc(sizeof(*tmp));
 if (tmp)
-  memset(tmp, 0, sizeof(*tmp));
+       memset(tmp, 0, sizeof(*tmp));
 
 This eliminates duplication of the 'struct foo' identifier, which makes the
 code easier to read and also ensures that if it is copy-and-pasted it won't