kenmirrors/asterisk
1
0
Fork 0
mirror of https://github.com/asterisk/asterisk.git synced 2025-10-13 00:04:53 +00:00
Code Issues Packages Projects Releases Wiki Activity

Add a janitor project to use ARRAY_LEN instead of in-line

Browse Source 
sizeof() and division.



git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@127566 65c4cc65-6c06-0410-ace0-fbb531ad65f3
This commit is contained in:
Mark Michelson
2008-07-02 21:09:18 +00:00
parent 3a27a6a9e7
commit 55e4daef88
1 changed files with 8 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
doc/janitor-projects.txt
View File

@@ -32,3 +32,11 @@
-- Audit all channel/res/app/etc. modules to ensure that they do not register any entrypoints with the Asterisk core until after they are ready to service requests; all config file reading/processing, structure allocation, etc. must be completed before Asterisk is made aware of any services the module offers.
-- Ensure that Realtime-enabled modules do not depend on the order of columns returned by the database lookup (example: outboundproxy and host settings in chan_sip).
-- There are several places in the code where the length of arrays is calculated in-line with sizeof() and division. A common place to find this is in for loops, like this:
for (i = 0; i < sizeof(array)/sizeof(array[0]); i++)
There is a macro in utils.h called ARRAY_LEN which should be used instead for readability's sake.
for (i = 0; i < ARRAY_LEN(array); i++)