[Home]

Summary:ASTERISK-05355: [patch] Update coding guidelines
Reporter:Olle Johansson (oej)Labels:
Date Opened:2005-10-23 11:14:20Date Closed:2008-01-15 15:53:01.000-0600
Priority:MinorRegression?No
Status:Closed/CompleteComponents: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