Summary: | ASTERISK-05355: [patch] Update coding guidelines | ||
Reporter: | Olle Johansson (oej) | Labels: | |
Date Opened: | 2005-10-23 11:14:20 | Date Closed: | 2008-01-15 15:53:01.000-0600 |
Priority: | Minor | Regression? | No |
Status: | Closed/Complete | Components: | Documentation |
Versions: | Frequency of Occurrence | ||
Related Issues: | |||
Environment: | Attachments: | ( 0) codelines.txt | |
Description: | This patch updates the layout of the coding guidelines and ads some text in the beginning and middle. The only change of the guidelines is the part about declaring variables in the beginning of a function for older GCC, that is now in the beginning of a code block. | ||
Comments: | By: Serge Vecher (serge-v) 2005-10-24 10:24:29 Spelling/clarity suggestions: 1) "... Code also need ..." -- "Code also needs" 2) "... and follow the ..." -- "and follows the" 3) "... separat patches ..." -- "separate patches" 4) "... Strings take are coming from input, should ..." -- "Strings that are coming from input should" 5) "... are just trying store ..." -- "are just trying to store" 6) "... Always derefrence ..." -- "Always dereference" 7) "... make it easier ..." -- "making it easier" By: Olle Johansson (oej) 2005-10-24 10:32:24 Thank you for correcting my swenglish, I'll look into it and update my patch. By: Olle Johansson (oej) 2005-10-24 10:42:10 Fixed speling erors ;-) By: Kevin P. Fleming (kpfleming) 2005-10-31 17:56:03.000-0600 Committed to CVS HEAD, thanks! By: Digium Subversion (svnbot) 2008-01-15 15:53:01.000-0600 Repository: asterisk Revision: 6909 U trunk/doc/CODING-GUIDELINES ------------------------------------------------------------------------ r6909 | kpfleming | 2008-01-15 15:53:00 -0600 (Tue, 15 Jan 2008) | 2 lines various text/formatting updates (issue ASTERISK-5355 plus some minor additions) ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=6909 |