diff options
| author | Steve Murphy <murf@digium.com> | 2008-10-09 14:17:33 +0000 |
|---|---|---|
| committer | Steve Murphy <murf@digium.com> | 2008-10-09 14:17:33 +0000 |
| commit | e235a073769eaf6e388bbe8d4eb5189db4018de6 (patch) | |
| tree | 3c963213e39ade4edd8366a127b3d3fe9d74e360 /doc/CODING-GUIDELINES | |
| parent | f6c78aa0feb3e13fc8e7262a650ab4c8aa4babe2 (diff) | |
(closes issue #13557)
Reported by: nickpeirson
Patches:
pbx.c.patch uploaded by nickpeirson (license 579)
replace_bzero+bcopy.patch uploaded by nickpeirson (license 579)
Tested by: nickpeirson, murf
1. replaced all refs to bzero and bcopy to memset and memmove instead.
2. added a note to the CODING-GUIDELINES
3. add two macros to asterisk.h to prevent bzero, bcopy from creeping
back into the source
4. removed bzero from configure, configure.ac, autoconfig.h.in
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@147807 65c4cc65-6c06-0410-ace0-fbb531ad65f3
Diffstat (limited to 'doc/CODING-GUIDELINES')
| -rw-r--r-- | doc/CODING-GUIDELINES | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/doc/CODING-GUIDELINES b/doc/CODING-GUIDELINES index 985df6846..ed320368b 100644 --- a/doc/CODING-GUIDELINES +++ b/doc/CODING-GUIDELINES @@ -382,6 +382,12 @@ processor operations, unlike ast_copy_string(). * Use of functions ------------------ +For the sake of uclibc, do not use index, bcopy or bzero; use +strchr(), memset(), and memmove() instead. uclibc can be configured +to supply these functions, but we can save these users +time and consternation if we abstain from using these +functions. + When making applications, always ast_strdupa(data) to a local pointer if you intend to parse the incoming data string. |