Boost logo

Boost :

Subject: Re: [boost] [detail] boost/detail/utf8_codecvt_facet.hpp broken by recent change
From: Beman Dawes (bdawes_at_[hidden])
Date: 2017-08-28 12:03:57

On Fri, Aug 25, 2017 at 12:39 PM, Robert Ramey via Boost <
boost_at_[hidden]> wrote:

> On 8/25/17 9:18 AM, Robert Ramey via Boost wrote:
>> On 8/25/17 6:09 AM, Beman Dawes via Boost wrote:
>>> boost/detail/utf8_codecvt_facet.hpp fails to compile for STL's MinGW
>>> distro from
>>> The error message is:
>>> ..\..\../boost/detail/utf8_codecvt_facet.hpp:117:29: error: function
>>> 'boost::filesystem::detail::utf8_codecvt_facet::~utf8_codecvt_facet()'
>>> definition is marked dllimport
>>> virtual BOOST_UTF8_DECL ~utf8_codecvt_facet(){}
>>> The problem (on develop) is caused by Robert Ramey's Aug 13 commit
>>> 73c12230c31a851c15ecb9a649468425ebd92f8e "adjustment to avoid linking
>>> problem: vtable not found found for ~utf8_codecvt_facet.hpp" which added
>>> "BOOST_UTF8_DECL" to line 117.
>>> This is breaking my local filesystem tests for GCC/MinGW. If the change
>>> is
>>> reverted, the tests start passing again.
>>> Robert, could you please revert the change? I can also to it, but thought
>>> you needed to be alerted first so you can revisit whatever you were
>>> trying
>>> to fix.
>> OK I see this. What I don't see is why it fails which I think is
>> important. I'd like a couple of days to try to understand why this is
>> happening.
> OK - I took a short look at this. It's clear that I can't figure it out
> in a reasonable time so the best would be to back out the change. Feel
> free to do it.
> <>



Boost list run by bdawes at, gregod at, cpdaniel at, john at