diff options
| author | kpfleming <kpfleming@f38db490-d61c-443f-a65b-d21fe96a405b> | 2005-04-01 04:38:12 +0000 |
|---|---|---|
| committer | kpfleming <kpfleming@f38db490-d61c-443f-a65b-d21fe96a405b> | 2005-04-01 04:38:12 +0000 |
| commit | 68428cbd9d12b335eee19b3f5ed9213992576cd4 (patch) | |
| tree | e295e0da2756e95a8e8d8f93fe20b41fa0501b11 /doc/CODING-GUIDELINES | |
| parent | 336490ff75d8e7946b7ba9914773ccdec0bd9adf (diff) | |
Update README to reflect modern Asterisk features and requirements
Add note in UPGRADE.txt about compiler requirements
Add note to CODING-GUIDELINES about new policy for CLI command structure
git-svn-id: http://svn.digium.com/svn/asterisk/trunk@5335 f38db490-d61c-443f-a65b-d21fe96a405b
Diffstat (limited to 'doc/CODING-GUIDELINES')
| -rwxr-xr-x | doc/CODING-GUIDELINES | 12 |
1 files changed, 11 insertions, 1 deletions
diff --git a/doc/CODING-GUIDELINES b/doc/CODING-GUIDELINES index 175fb5380..f115094d0 100755 --- a/doc/CODING-GUIDELINES +++ b/doc/CODING-GUIDELINES @@ -78,7 +78,6 @@ for (x=0;x<5;x++) { } - Make sure you never use an uninitialized variable. The compiler will usually warn you if you do so. @@ -154,3 +153,14 @@ if (name && (len = strlen(name) + strlen(prefix) + strlen(postfix) + 3) && (newn Use const on pointers which your function will not be modifying, as this allows the compiler to make certain optimizations. + +== CLI Commands == + +New CLI commands should be named using the module's name, followed by a verb +and then any parameters that the command needs. For example: + +*CLI> iax2 show peer <peername> + +not + +*CLI> show iax2 peer <peername> |