Change ast_hangup() to return void and be NULL safe.
authorRichard Mudgett <rmudgett@digium.com>
Wed, 17 Jul 2013 22:30:28 +0000 (22:30 +0000)
committerRichard Mudgett <rmudgett@digium.com>
Wed, 17 Jul 2013 22:30:28 +0000 (22:30 +0000)
commit40ce5e0d18520cd7a440cd8849e3e920be68f471
tree5499af489ac45954318119e4294811be3da06dff
parentda1902cdc060763a712bb1586cfca5ec1e9a2c49
Change ast_hangup() to return void and be NULL safe.

Since ast_hangup() is effectively a channel destructor, it should be a
void function.

* Make the few silly callers checking the return value no longer do so.
Only the CDR and CEL unit tests checked the return value.

* Make all callers take advantage of the NULL safe change and remove the
NULL check before the call.

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@394623 65c4cc65-6c06-0410-ace0-fbb531ad65f3
12 files changed:
addons/chan_mobile.c
apps/app_meetme.c
apps/app_voicemail.c
include/asterisk/channel.h
main/channel.c
main/dial.c
main/features.c
tests/test_app.c
tests/test_cdr.c
tests/test_cel.c
tests/test_stasis_endpoints.c
tests/test_voicemail_api.c