diff options
author | Kevin P. Fleming <kpfleming@digium.com> | 2005-04-23 15:11:28 +0000 |
---|---|---|
committer | Kevin P. Fleming <kpfleming@digium.com> | 2005-04-23 15:11:28 +0000 |
commit | f9876ccfa86bafd9e31cac20900c2e007b3b1891 (patch) | |
tree | 58271a4ca57826f59f61996c933d7d24013c2e4d /doc/CODING-GUIDELINES | |
parent | fd4dbd6c5c002398a3e893ebf2ee4cbecfa9e8cc (diff) |
add guideline comment about not using '%i' with scanf
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@5501 65c4cc65-6c06-0410-ace0-fbb531ad65f3
Diffstat (limited to 'doc/CODING-GUIDELINES')
-rwxr-xr-x | doc/CODING-GUIDELINES | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/doc/CODING-GUIDELINES b/doc/CODING-GUIDELINES index f115094d0..b9fcfba1e 100755 --- a/doc/CODING-GUIDELINES +++ b/doc/CODING-GUIDELINES @@ -24,6 +24,10 @@ Try to match the existing formatting of the file you are working on. Functions and variables that are not intended to be global must be declared static. +When reading integer numeric input with scanf (or variants), do _NOT_ use '%i' +unless specifically want to allow non-base-10 input; '%d' is always a better +choice, since it will not silently turn numbers with leading zeros into base-8. + Roughly, Asterisk coding guidelines are generally equivalent to the following: |