-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4535/
-----------------------------------------------------------
(Updated March 29, 2015, 9:39 p.m.)
Status
------
This change has been marked as submitted.
Review request for Asterisk Developers.
Changes
-------
Committed in revision 433747
Bugs: ASTERISK-24917
https://issues.asterisk.org/jira/browse/ASTERISK-24917
Repository: Asterisk
Description
-------
clang's static analyzer will throw quite a number warnings / errors during
compilation, some of which can be very helpfull in finding corner-case bugs
clang compiler warning:-Wenum-conversion
Changes:
/branches/13/channels/chan_pjsip.c:923
Wrong enum being used
/branches/13/channels/chan_sip.c:19102
Incorrect enum
/branches/13/channels/chan_sip.c:19107
Incorrect enum returned
/branches/13/include/asterisk/strings.h:1236
Replaced enum ao2_container_opts opts -> enum ao2_alloc_opts opts
Not 100% sure if this substition is correct. Please verify.
/branches/13/main/strings.c:173
Replaced enum ao2_container_opts opts -> enum ao2_alloc_opts opts
Not 100% sure if this substition is correct. Please verify.
/branches/13/res/res_stasis.c:1803
Incorrect enum for return value
Diffs
-----
/branches/13/res/res_stasis.c 433444
/branches/13/main/strings.c 433444
/branches/13/include/asterisk/strings.h 433444
/branches/13/channels/chan_sip.c 433444
/branches/13/channels/chan_pjsip.c 433444
Diff: https://reviewboard.asterisk.org/r/4535/diff/
Testing
-------
Thanks,
Diederik de Groot
--
_____________________________________________________________________
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
http://lists.digium.com/mailman/listinfo/asterisk-dev
