----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4540/#review14938 -----------------------------------------------------------
Ship it! Ship It! - Matt Jordan On March 27, 2015, 6:51 a.m., Diederik de Groot wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/4540/ > ----------------------------------------------------------- > > (Updated March 27, 2015, 6:51 a.m.) > > > Review request for Asterisk Developers. > > > 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
