Boost C++ Libraries: Ticket #12917: string_ref not constructible from rvalue strings anymore

Marshall Clow — Mon, 20 Mar 2017 13:03:36 GMT

Can you please give an (non-contrived) example of user code that is broken by this?

arvid.norberg@… — Mon, 20 Mar 2017 23:20:02 GMT

one example of this from my project:

std::string combine_path(string_view left, string_view right);

It used to be possible to nest these, like this:

auto a = combine_path(foo, combine_path(bar, b));

in boost-1.64 this is no longer possible, a proposed work-around is to introduce a string_view make_sv(std::string const&) function, to turn the temporary into an lvalue and construct the string_view from that. https://github.com/arvidn/libtorrent/pull/1814

Marshall Clow — Tue, 21 Mar 2017 04:29:34 GMT

thanks for the example. The change was made in response to https://svn.boost.org/trac/boost/ticket/11740. "string_ref not working correctly with rvalue of std::string"

olafvdspek@… — Wed, 22 Mar 2017 14:27:17 GMT

https://github.com/mclow/String-Algo/issues/8

return db.query(make_query(db, q, a...));

make_query returns string, db.query takes string_view.