Project

General

Profile

Actions

Bug #7002

closed

Compile failures with char8_t by MSVC under /std:c++latest(C++20) mode

Added by Quella Zhang about 5 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Roel Standaert
Target version:
Start date:
04/16/2019
Due date:
% Done:

100%

Estimated time:

Description

We've stumbled across some build failures in Wt after implementing support for char8_t under /std:c++latest(C20) in the development version of Visual C++. Could you help look at this? Thanks in advance! Noted that this issue only found when compiles with unreleased vctoolset, that next release of MSVC will have this behavior.

Note:

Wt requires boost. You can get boost 32bits and 64bits from following address and Change the install path:

32bits: c:\boost_1_67_0\x86

67bits: c:\boost_1_67_0\x67

Boost: https://sourceforge.net/projects/boost/files/boost-binaries

Repro steps:

  1. open VS2017 x86 Native tools command tool
  2. git clone https://github.com/emweb/wt d:\Wt\src
  3. cd D:\Wt
  4. mkdir build_x86 && pushd build_x86
  5. cmake -G "Visual Studio 15 2017" -DCMAKE_SYSTEM_VERSION=10.0.17134.0 -DBOOST_PREFIX=C:\boost_1_67_0\x86 -DBOOST_LIBRARYDIR=C:\boost_1_67_0\x86\lib32-msvc-14.1 ..\src\
  6. msbuild /m /p:Configuration=Release;Platform=Win32 ALL_BUILD.vcxproj /t:Rebuild

Failures:

D:\Wt\src\test\utf8\Utf8Test.C (22): error C2440: 'initializing': cannot convert from 'const char8_t [42]' to 'std::basic_string<char,std::char_traits,std::allocator>'

D:\Wt\src\test\utf8\Utf8Test.C (22): note: No constructor could take the source type, or constructor overload resolution was ambiguous

D:\Wt\src\test\utf8\Utf8Test.C (145): error C2440: 'initializing': cannot convert from 'const char8_t [42]' to 'std::basic_string<char,std::char_traits,std::allocator>'

D:\Wt\src\test\utf8\Utf8Test.C (145): note: No constructor could take the source type, or constructor overload resolution was ambiguous

D:\Wt\src\test\utf8\Utf8Test.C (157): error C2665: 'Wt::utf8': none of the 2 overloads could convert all the argument types

D:\Wt\src\src\Wt/WString.h(778): note: could be 'Wt::WString Wt::utf8(const std::string &)'

D:\Wt\src\src\Wt/WString.h(772): note: or 'Wt::WString Wt::utf8(const char *)'

D:\Wt\src\test\utf8\Utf8Test.C (157): note: while trying to match the argument list '(const char8_t [42])'

Actions #1

Updated by Roel Standaert about 5 years ago

I see. The C++20 standard proposal is kind of breaking backwards compatibility there. This may go rather deep, so we'll have to see what we should do there. It would make sense that WString::toUTF8() returns std::u8string instead of std::string, for example. Then there's all of the use cases where we use a WString not for i18n purposes (WString::tr and friends), but only because that implies UTF-8 encoding. It would make sense to take a std::u8string from C++20 on, instead.

Actions #2

Updated by Roel Standaert about 3 years ago

  • Description updated (diff)
  • Status changed from New to InProgress
  • Assignee set to Roel Standaert
  • Target version set to 4.5.1
Actions #3

Updated by Roel Standaert about 3 years ago

  • Description updated (diff)
Actions #4

Updated by Roel Standaert about 3 years ago

  • Status changed from InProgress to Resolved
Actions #5

Updated by Roel Standaert over 2 years ago

  • % Done changed from 0 to 100
Actions #6

Updated by Roel Standaert over 2 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF