Boost C++ Libraries: Ticket #6854: boost/random/detail/integer_log2.hpp:71:35: warning: always_inline function might not be inlinable

status changed; resolution set

Steven Watanabe — Sat, 19 May 2012 13:49:02 GMT

status new → closed
resolution → fixed

(In [78513]) Add missing inline. Fixes #6854.

cc set

Thu, 28 Jun 2012 15:04:18 GMT

cc mika.fischer@… added

anonymous — Sun, 15 Jul 2012 21:39:20 GMT

Why has this not landed in 1.50.0?

benjamin.mahler@… — Sun, 09 Sep 2012 18:04:55 GMT

Replying to anonymous:

Why has this not landed in 1.50.0?

Does not appear to be in 1.51.0 either..

gromer@… — Wed, 07 Nov 2012 00:40:04 GMT

Replying to benjamin.mahler@…:

Replying to anonymous:

Why has this not landed in 1.50.0?

Does not appear to be in 1.51.0 either..

Nor in 1.52.0. What gives?

status, version, severity changed; resolution deleted

anonymous — Thu, 15 Nov 2012 17:09:03 GMT

status closed → reopened
version Boost 1.49.0 → Boost 1.52.0
resolution fixed
severity Cosmetic → Regression

Indeed, still not fixed in 1.52, reopening this ticket.

Adding the suggested inline keyword to integer_log2.hpp fixes this.

/tmp/boost_1_52_0$ /usr/bin/g++-4.7 -I./ -Werror main.cpp
In file included from ./boost/random/detail/large_arithmetic.hpp:19:0,
                 from ./boost/random/detail/const_mod.hpp:23,
                 from ./boost/random/detail/seed_impl.hpp:27,
                 from ./boost/random/mersenne_twister.hpp:29,
                 from main.cpp:1:
./boost/random/detail/integer_log2.hpp:71:35: error: always_inline function
might not be inlinable [-Werror=attributes]
cc1plus: all warnings being treated as errors

version changed

tdmackey@… — Thu, 24 Jan 2013 19:33:14 GMT

version Boost 1.52.0 → Boost 1.53.0

This is still not fixed in 1.53.0 beta 1.

status changed; resolution set

Marshall Clow — Fri, 25 Jan 2013 17:06:10 GMT

status reopened → closed
resolution → fixed

(In [82609]) Merge bug fix to release; Fixes #6854