Subject: Re: [boost] [detail] boost/detail/utf8_codecvt_facet.hpp broken by recent change
From: Robert Ramey (ramey_at_[hidden])
Date: 2017-08-25 16:39:02
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 https://nuwen.net/mingw.html
>>
>> 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.
Robert Ramey