Force non-inlining of ao2_iterator_destroy when TEST_FRAMEWORK is enabled
authorMatthew Jordan <mjordan@digium.com>
Wed, 14 Mar 2012 14:40:44 +0000 (14:40 +0000)
committerMatthew Jordan <mjordan@digium.com>
Wed, 14 Mar 2012 14:40:44 +0000 (14:40 +0000)
commit6df0ae5c1fb6b780874f859e9c1be89c5ee881ac
treed04fc4cc05cdbd6dcd889d95ebc198a2b72cf84f
parent00b270833f3d5a58ab2acbcce1c5a16b0330aaeb
Force non-inlining of ao2_iterator_destroy when TEST_FRAMEWORK is enabled

In r357272, astobj2 was changed to automatically enable REF_DEBUG when the
TEST_FRAMEWORK flag was enabled.  Unfortunately, some compilers (gcc 4.5.1
at least) will attempt to inline ao2_iterator_destroy in handle_astobj2_test.
This by itself is not a problem; unfortunately, the compiler believes that
there is a code path wherein an object allocated on the stack will be
free'd.  As warnings are treated as errors, this prevents compilation of
astobj2.

This patch works around that by adding the noinline attribue to
ao2_iterator_destroy, but only if the TEST_FRAMEWORK flag is enabled.
Preventing inlining is only needed for the test method defined in astobj2,
which is also only enabled if TEST_FRAMEWORK is enabled.

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@359306 65c4cc65-6c06-0410-ace0-fbb531ad65f3
include/asterisk/astobj2.h