There is still a lot of similarity with builtin_blindtransfer()
which should be removed by definining functions for the common
pieces of code (eg in the first part).
git-svn-id: http://svn.digium.com/svn/asterisk/trunk@21097 f38db490-d61c-443f-a65b-d21fe96a405b
Please place a Doxygen todo marker before things that needs to be checked up later
by someone - it's like /*! \todo We really need to implement this in C++ */
- option_debug checking before logging to DEBUG channel
git-svn-id: http://svn.digium.com/svn/asterisk/trunk@20790 f38db490-d61c-443f-a65b-d21fe96a405b
replacing some common blocks of code (there are still a few
instances where it can be used)
git-svn-id: http://svn.digium.com/svn/asterisk/trunk@20598 f38db490-d61c-443f-a65b-d21fe96a405b
- replace nested 'if' with '&&'
- fix indentation
- annotate the end brace of some huge block
- remove useless parentheses
git-svn-id: http://svn.digium.com/svn/asterisk/trunk@20597 f38db490-d61c-443f-a65b-d21fe96a405b
removing replicated var = var->next;
- remove a potential infinite loop and document the problem
- remove useless checks and document why
- mark XXX a possible bug (to be investigated)
- use ast_strlen_zero() instead of expanding it inline
- fix indentation in one place
- replace a nested 'if' with '&&'
git-svn-id: http://svn.digium.com/svn/asterisk/trunk@20223 f38db490-d61c-443f-a65b-d21fe96a405b
- add braces around a nested 'if'
- use S_OR and '?' to remove some duplicated function calls;
- replace nested 'if' with &&
- move out a common term in a sequence of 'if'
- add a comment on a potentially dangerous string manipulation
git-svn-id: http://svn.digium.com/svn/asterisk/trunk@20196 f38db490-d61c-443f-a65b-d21fe96a405b
As partly documented in loader.c and include/asterisk/module.h,
modules are now expected to return all of their methods and flags
into a structure 'mod_data', and are normally loaded with RTLD_NOW
| RTLD_LOCAL, so symbols are resolved immediately and conflicts
should be less likely. Only in a small number of cases (res_*,
typically) modules are loaded RTLD_GLOBAL, so they can export
symbols.
The core of the change is only the two files loader.c and
include/asterisk/module.h, all the rest is simply adaptation of the
existing modules to the new API, a rather mechanical (but believe
me, time and finger-consuming!) process whose detail you can figure
out by svn diff'ing any single module.
Expect some minor compilation issue after this change, please
report it on mantis http://bugs.digium.com/view.php?id=6968
so we collect all the feedback in one place.
I am just sorry that this change missed SVN version number 20000!
git-svn-id: http://svn.digium.com/svn/asterisk/trunk@20003 f38db490-d61c-443f-a65b-d21fe96a405b
constant string into a buffer that we know is big enough, don't use a length
limited copy at all, use strcpy.
... as stated in the coding guidelines.
git-svn-id: http://svn.digium.com/svn/asterisk/trunk@18721 f38db490-d61c-443f-a65b-d21fe96a405b
nested statements in a block without branches, which is a violation of the
coding guidelines.
As a matter of fact, this module violates the coding guidelines in multiple
ways that including formatting and code issues. In my opinion, this module
should not have been merged into the trunk in this form.
git-svn-id: http://svn.digium.com/svn/asterisk/trunk@18693 f38db490-d61c-443f-a65b-d21fe96a405b
- implement all functions through internal APIs in res_osp.c and app_osplookup.c
(homesick)
git-svn-id: http://svn.digium.com/svn/asterisk/trunk@18369 f38db490-d61c-443f-a65b-d21fe96a405b