misc. code cleanup

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@20169 65c4cc65-6c06-0410-ace0-fbb531ad65f3

code simplifications

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@20168 65c4cc65-6c06-0410-ace0-fbb531ad65f3

use ast_strdup instead of strdup

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@20167 65c4cc65-6c06-0410-ace0-fbb531ad65f3

normalize cli completion code

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@20166 65c4cc65-6c06-0410-ace0-fbb531ad65f3

constification and code simplifications

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@20165 65c4cc65-6c06-0410-ace0-fbb531ad65f3

constify a couple of function arguments

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@20138 65c4cc65-6c06-0410-ace0-fbb531ad65f3

code cleanup (for loop normalization, remove useless casts and parentheses)

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@20132 65c4cc65-6c06-0410-ace0-fbb531ad65f3

partial fix of the module API to use the new method.
Still not complete as we need to take care of the usecount stuff.

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@20104 65c4cc65-6c06-0410-ace0-fbb531ad65f3

fix another svn merge bug

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@20076 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Compatibility fixes for loader changes

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@20075 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 20037 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.2

........
r20037 | kpfleming | 2006-04-14 12:21:09 -0500 (Fri, 14 Apr 2006) | 2 lines

uncomment files that actually do exist (oops)

........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@20038 65c4cc65-6c06-0410-ace0-fbb531ad65f3

fix a mostly harmless error introduced by svn merge.

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@20036 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 20034 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.2

........
r20034 | kpfleming | 2006-04-14 12:00:32 -0500 (Fri, 14 Apr 2006) | 2 lines

update text to match actual prompts being distributed (thanks to Kinsey in the support department for reviewing all the prompts!)

........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@20035 65c4cc65-6c06-0410-ace0-fbb531ad65f3

This rather large commit changes the way modules are loaded.

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: https://origsvn.digium.com/svn/asterisk/trunk@20003 65c4cc65-6c06-0410-ace0-fbb531ad65f3

temporarily revert the way mutex are initialized on BSD systems.
This fixes the compilation on OS/X (the change exposed a wrong
assumption on mutex types on OS/X), but still leaves open the
bugs in initializing mutex on bsd systems, which you will see
reported as 'locking failures' on certain operations.
I need to investigate the issue further, but the best thing
i can do now is leave things as they have been for months.

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19973 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Bug 6927 - CLI command has 3 args, not 2.

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19941 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 19891 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.2

........
r19891 | tilghman | 2006-04-13 15:37:50 -0500 (Thu, 13 Apr 2006) | 2 lines

Bug 6947 - Allow vm broadcasts to more than 256 characters worth of mailboxes

........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19892 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Fix format for big endian systems

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19890 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Fix the grammar that Kevin broke

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19858 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Bug 6896 - Fix for garbled wav49 format

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19814 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 19812 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.2

........
r19812 | kpfleming | 2006-04-13 12:40:21 -0500 (Thu, 13 Apr 2006) | 2 lines

oops... let's not set a variable and then immediately overwrite it while assuming its old value will magically return

........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19813 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Unregister format says it's already unregistered

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19811 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Typo fix

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19810 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Blocked revisions 19768 via svnmerge

........
r19768 | tilghman | 2006-04-13 10:56:35 -0500 (Thu, 13 Apr 2006) | 2 lines

Bug 6957 - variable names beginning with CALLERID weren't substituted correctly

........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19769 65c4cc65-6c06-0410-ace0-fbb531ad65f3

remove extraneous warning message

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19767 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Issue #6951 - Show last successful registration time in "sip show registry" (ivanfm) with mods for svn trunk

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19736 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Formatting fixes

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19703 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Document the MSGSM format, and fix the uncalculated number of samples

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19673 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Updates to speech recognition API and dialplan utilities. Moved to using dialplan functions, and some other misc things.

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19645 65c4cc65-6c06-0410-ace0-fbb531ad65f3

fix caffiene-induced typo.

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19616 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Note that the res_speech module will need to be loaded first, and add a conveient line to uncomment to do so for the time being.

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19615 65c4cc65-6c06-0410-ace0-fbb531ad65f3

simplify macro usage

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19614 65c4cc65-6c06-0410-ace0-fbb531ad65f3

make sure uint64_t is available on mac

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19579 65c4cc65-6c06-0410-ace0-fbb531ad65f3

- Adding some refer state definitions from the siptransfer branch
- A bit more URI conversions
- Changes to sip_dual and the SIP invite structure
- Add Supported: headers to more requests

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19578 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Fix for Polycom bug...

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19549 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Implement a setting for denying/allowing transfer requests. At this stage,
we only have open/closed. Well, at least you can deny transfers from unknown
callers or at least incoming calls.
(Part of the SIPtransfer branch)

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19546 65c4cc65-6c06-0410-ace0-fbb531ad65f3

whoops... remove an extra line.

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19545 65c4cc65-6c06-0410-ace0-fbb531ad65f3

add 'show threads' and 'show profile' commands.
These are momstly debugging tools for developers,
a bit documented in the header files (utils.h),
although more documentation is definitely necessary.

The performance impact is close to zero(*) so there is no
need to compile it conditionally.
(*) not completely true - thread destruction still needs
to search a list _but_ this can be easily optimized if we
end up with hundreds of active threads (in which case, though,
the problem is clearly elsewhere).

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19544 65c4cc65-6c06-0410-ace0-fbb531ad65f3

fancify the sample grammar a bit

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19543 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Doxygen docs

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19542 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Add ability to see if the person calling said anything or not.

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19512 65c4cc65-6c06-0410-ace0-fbb531ad65f3

reindent block properly

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19466 65c4cc65-6c06-0410-ace0-fbb531ad65f3

localize some variables, remove useless parentheses

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19465 65c4cc65-6c06-0410-ace0-fbb531ad65f3

remove useless \0, and fix formatting.

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19437 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Bug 6943 - transition away from using CallerID header, when we really mean CallerIDNum

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19436 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 19397 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.2

........
r19397 | tilghman | 2006-04-11 17:39:59 -0500 (Tue, 11 Apr 2006) | 2 lines

Bug 6490 - telco intercept should report NOANSWER instead of CHANUNAVAIL

........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19398 65c4cc65-6c06-0410-ace0-fbb531ad65f3

fix various bugs in the DEBUG_THREADS code including:
- misspelled ast_mutex_logger() instead of __ast_mutex_logger()
- misplaced #define ast_mutex_init(pmutex)
- wrong arguments to __ast_mutex_logger() in one instance.

Clearly this code is too spaghetti!

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19396 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 19394 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.2

........
r19394 | tilghman | 2006-04-11 17:24:46 -0500 (Tue, 11 Apr 2006) | 2 lines

Bug 6061 - Fix ODBC storage of VM on PGSQL and MSSQL

........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19395 65c4cc65-6c06-0410-ace0-fbb531ad65f3

fix logic error; don't test for rtcache flag unless asked to (issue #6923)

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19393 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 19353 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.2

........
r19353 | kpfleming | 2006-04-11 16:58:47 -0500 (Tue, 11 Apr 2006) | 2 lines

don't create a 'voicemail' symlink in the sounds directory; app_voicemail has not needed it since January of 2005 (issue #6613)

........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19354 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 19351 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.2

........
r19351 | tilghman | 2006-04-11 16:55:51 -0500 (Tue, 11 Apr 2006) | 2 lines

Bug 6097 - possible descriptor leak

........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19352 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 19348 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.2

........
r19348 | kpfleming | 2006-04-11 16:50:18 -0500 (Tue, 11 Apr 2006) | 2 lines

don't call the originating device as part of the Page() operation (issue #6932)

........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19350 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 19347 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.2

........
r19347 | kpfleming | 2006-04-11 16:35:52 -0500 (Tue, 11 Apr 2006) | 2 lines

simplify spy queue flushing logic, and always force a flush when one side gets full, even if the other side is not empty (issue #6457)

........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19349 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 19345 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.2

........
r19345 | kpfleming | 2006-04-11 16:14:42 -0500 (Tue, 11 Apr 2006) | 2 lines

don't destroy the entire dialplan during 'reload', just atomically replace it like 'extensions reload' does (issue #6047)

........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19346 65c4cc65-6c06-0410-ace0-fbb531ad65f3

remove unused variable (discovered by the compiler)

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19306 65c4cc65-6c06-0410-ace0-fbb531ad65f3

No need to do this in here any longer since the linkedlists macro is fixed

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19305 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 19303 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.2

........
r19303 | file | 2006-04-11 17:46:38 -0300 (Tue, 11 Apr 2006) | 2 lines

Minor linked lists bug fix. When you're dealing with swapping entries around a lot it can cause a seg fault.

........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19304 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 19301 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.2

........
r19301 | kpfleming | 2006-04-11 15:11:01 -0500 (Tue, 11 Apr 2006) | 2 lines

handle call time limit properly when warning is requested _after_ call would hae already ended (issue #6356)

........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19302 65c4cc65-6c06-0410-ace0-fbb531ad65f3

typo

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19272 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Convert chan_iax2 to use linked lists for multithreading, and add dynamic threads. These are used when all pool threads are in use, and will stick around until load dies down. The theory is that during high load you'll have more threads available, and during low load you'll only have the normal pool threads sticking around.

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19254 65c4cc65-6c06-0410-ace0-fbb531ad65f3

 Minor cleanups and error handling for app_dial #6935 (casper)

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19253 65c4cc65-6c06-0410-ace0-fbb531ad65f3

remove an unused function

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19226 65c4cc65-6c06-0410-ace0-fbb531ad65f3

as discussed with kevin, move
ast_register_atexit()/ ast_unregister_atexit() into asterisk.h
These are general functions, not restricted to modules, so move
them in a more proper place.

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19223 65c4cc65-6c06-0410-ace0-fbb531ad65f3

use proper type

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19222 65c4cc65-6c06-0410-ace0-fbb531ad65f3

 Provide the ability to read extension number after name #6938 (delink)

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19221 65c4cc65-6c06-0410-ace0-fbb531ad65f3

normalize code preparing for loader changes

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19220 65c4cc65-6c06-0410-ace0-fbb531ad65f3

remove unused fields

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19219 65c4cc65-6c06-0410-ace0-fbb531ad65f3

normalize code in preparation to module changes

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19189 65c4cc65-6c06-0410-ace0-fbb531ad65f3

staticize a function, and normalize code in preparation to module changes.

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19188 65c4cc65-6c06-0410-ace0-fbb531ad65f3

mISDN Messages must be freed with free_msg \!\!

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19160 65c4cc65-6c06-0410-ace0-fbb531ad65f3

- First stab at removing debug and ignore variables that
  we pass along function calls, instead implementing them as
  flags on the incoming packet.
- Adding forward declarations of handle_request functions

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19128 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Add new documentation files to index file 00README.1st and to doxygen

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19097 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Bugs 6883, 6930: compiler warnings

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19095 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Data stores do not need a lock. As well change the way they are removed from the channel when it is destroyed (thanks Russell Wussell) and finally... because C++ is silly... change our list macro info thing to be "entry" instead of "list".

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19062 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 19008 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.2

........
r19008 | bweschke | 2006-04-10 20:05:01 -0500 (Mon, 10 Apr 2006) | 3 lines

 When using the silence detector in ast_play_and_record() and ast_play_and_prepend(), the truncation code never gets called to remove the detected silence, because the value of res is zero when control gets to that point. #6903 w/some mods (softins)

........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@19009 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Presenting a revised data stores and oh my, a generic speech recognition API! I wonder what we can do with this now...

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@18979 65c4cc65-6c06-0410-ace0-fbb531ad65f3

remove support for BYEXTENSION (which nobody even knows about anymore)

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@18977 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Fixup fixup - add some debugging and error handling

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@18940 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Resolve conflicts, prepare for next batch of conflicts

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@18909 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 18866 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.2

........
r18866 | bweschke | 2006-04-10 12:29:51 -0500 (Mon, 10 Apr 2006) | 2 lines

 Don't say that we can pass an 'exten' argument in the documentation of Park() when we really cannot. #6902 (opsys)

........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@18867 65c4cc65-6c06-0410-ace0-fbb531ad65f3

fixed some issues, that appear at higher load

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@18835 65c4cc65-6c06-0410-ace0-fbb531ad65f3

we send nearly everytime a RELEASE, only if we for sure know, that it's a TE and we did create the call we don't to hear the Inband Info

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@18800 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Making sure that cancel destroy is only executed once...

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@18799 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Small fix

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@18797 65c4cc65-6c06-0410-ace0-fbb531ad65f3

- staticize gettag() complete_sip_peer() get_calleridname()  arguments;
- use strsep() instead of strchr() where appropriate
- constify some args and add comments.
- remove a conditional near line 1940 - we already know what to use.

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@18793 65c4cc65-6c06-0410-ace0-fbb531ad65f3

- constification of some functions (args and return values):
        transmit_response_reliable()
        hangup_cause2sip()
- remove useless braces;
- add comments on some slightly cryptic code segments
- mark XXX possible critical pieces of code.
- remove an unneeded string termination after ast_copy_string
- mark usage of some rarely used functions;
- use strsep() instead of emulating it inline;
- replace magic constants with sizeof(array)

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@18792 65c4cc65-6c06-0410-ace0-fbb531ad65f3

- localize 'struct cfalias' into the only function using it;
- remove duplicate code to walk through sdp packets, replacing
  sdpLineNum_iterator_init(&foo); with  "foo = 0";
- remove duplicate code to test ast_test_flag(&p->flags[0], SIP_NAT_ROUTE);

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@18791 65c4cc65-6c06-0410-ace0-fbb531ad65f3

constify get_sdp*() and friends.

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@18790 65c4cc65-6c06-0410-ace0-fbb531ad65f3

constify get_header(), which let me find out and fix one bug (overwriting
a string in the buffer) and finding out another one (not fixed yet,
just marked XXX).

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@18789 65c4cc65-6c06-0410-ace0-fbb531ad65f3

- describe how sip packets are stored internally;
- remove useless braces or local variables;
- simplify some code sequences;
- mark with XXX a possible locking issue.

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@18788 65c4cc65-6c06-0410-ace0-fbb531ad65f3

struct sip_request cleanup:
- remove a debug field that was read but never set, so it was basically
  unused as well as the code testing it (also removed);

- make scalar fields contiguous so any array overflow will be
  less harmful;

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@18787 65c4cc65-6c06-0410-ace0-fbb531ad65f3

more cleanup: remove useless braces, replace "if" with "?",
localize a couple of variables, remove trailing whitespace.

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@18786 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Re-instate removed comment

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@18785 65c4cc65-6c06-0410-ace0-fbb531ad65f3

another batch of minor code simplifications
(moving repeated expressions into a function, const on
some arguments)

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@18784 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Bug 6829 - asprintf for Solaris

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@18755 65c4cc65-6c06-0410-ace0-fbb531ad65f3

remove a couple more unnecessary "out of memory" error messages

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@18723 65c4cc65-6c06-0410-ace0-fbb531ad65f3

use ast_*alloc and don't create duplicated error messages.

... as stated in the coding guidelines.

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@18722 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Instead of using strncpy, use ast_copy_string.  Also, in the case of copying a
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: https://origsvn.digium.com/svn/asterisk/trunk@18721 65c4cc65-6c06-0410-ace0-fbb531ad65f3

run the module through indent to fix the formatting issues that violate the
coding guidelines

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@18704 65c4cc65-6c06-0410-ace0-fbb531ad65f3

fix typo in the provided indent command

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@18694 65c4cc65-6c06-0410-ace0-fbb531ad65f3