-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4540/
-----------------------------------------------------------

(Updated April 6, 2015, 12:52 p.m.)


Status
------

This change has been marked as submitted.


Review request for Asterisk Developers.


Changes
-------

Committed in revision 434087


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:-Wformat


Issue:
framehook.c:141:16: warning: format specifies type 'unsigned short' but the 
argument has type 'int' [-Wformat]
                        i->version, AST_FRAMEHOOK_INTERFACE_VERSION);
                                    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/data/development/asterisk/asterisk-13-branch/include/asterisk/framehook.h:227:41:
 note: expanded from macro 'AST_FRAMEHOOK_INTERFACE_VERSION'
#define AST_FRAMEHOOK_INTERFACE_VERSION 4

Changed format: to use %i instead of %hu


Diffs
-----

  /branches/13/main/framehook.c 433444 

Diff: https://reviewboard.asterisk.org/r/4540/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

Reply via email to