Re: [asterisk-dev] [Code Review] 4540: clang compiler warning: -Wformat
--- 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
Re: [asterisk-dev] [Code Review] 4540: clang compiler warning: -Wformat
--- 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
[asterisk-dev] [Code Review] 4540: clang compiler warning: -Wformat
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4540/ --- 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 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
Re: [asterisk-dev] [Code Review] 4540: clang compiler warning: -Wformat
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4540/ --- (Updated March 27, 2015, 12:51 p.m.) Review request for Asterisk Developers. Bugs: ASTERISK-24917 https://issues.asterisk.org/jira/browse/ASTERISK-24917 Repository: Asterisk Description (updated) --- 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