Boost C++ Libraries: Ticket #6212: "Method 3.1: Implement a Range Adaptor without arguments" does not implement

ne01026@… — Mon, 05 Dec 2011 14:44:25 GMT

It turns out that the following code is needed to finish up:

namespace
        {
            const detail::reverse_forwarder reversed =
                                            detail::reverse_forwarder();
        }

I wonder why the explicit assignment is needed? Or should it be a reference instead?

Nathan Ridge — Wed, 02 Jan 2013 07:34:39 GMT

Replying to ne01026@…:

I wonder why the explicit assignment is needed? Or should it be a reference instead?

It should not be a reference (it would be a reference to a temporary which will have died long before one tries to use it).

Regarding why the explicit assignment is needed, see http://stackoverflow.com/questions/8092670/uninitialized-const. I believe

    const detail::reverse_forwarder reversed = {};

also does the trick.

status changed; resolution set

Nathan Ridge — Wed, 02 Jan 2013 07:51:24 GMT

status new → closed
resolution → fixed

(In [82314]) [range] fixed #6212 (missing step in documentation for how to write a range adaptor)

Nathan Ridge — Wed, 02 Jan 2013 07:53:47 GMT

Fixed in trunk. The fix will appear in 1.53 (I will merge to release branch after 1.53 beta release).

Thanks for the report!

Nathan Ridge — Mon, 14 Jan 2013 09:19:44 GMT

(In [82487]) [range] Merge documentation fixes and minor refactoring of docs (refs #5160; refs #5314; refs #5440; refs #5775; refs #6212; refs #7585; refs #7843; refs #7866).