Handle wchar_t as native C++11 type on windows

Desktop / LibreOffice - David Ostrovsky [ostrovsky.org] - 3 May 2016 14:26 UTC

The option /Zc:wchar_t- prevented to use wchar_t as a built-in type according to the C++ standard. In Visual C++ 6.0 and earlier, wchar_t was not implemented as a built-in type, but was declared in wchar.h as a typedef for unsigned short. Now, years later after the end of life this outdated toolchain, there is no reason not to use native type.

The only issue could be the ABI compatibility. But on a quick look at least, it looks like none of the mangled C++ symbols in the stable URE interface actually depend on wchar_t.

We forgot to get rid of /Zc:wchar_t- in 5.1. Do that for LibreOffice 5.2, though.

Change-Id: I8d6b380660859efa44c83c830734978d31d756a0 Reviewed-on: https://gerrit.libreoffice.org/22589

e16fa71 Handle wchar_t as native C++11 type on windows
cppu/qa/test_unotype.cxx | 60 --------------------------------
external/curl/ExternalProject_curl.mk | 2 +-
external/icu/UnpackedTarball_icu.mk | 1 -
external/icu/icu4c-wchar_t.patch | 16 ---------
include/cppu/unotype.hxx | 6 ++--
onlineupdate/Executable_updater.mk | 6 ----
solenv/gbuild/platform/com_MSC_defs.mk | 2 --
solenv/gcc-wrappers/wrapper.cxx | 1 -
8 files changed, 3 insertions(+), 91 deletions(-)

Upstream: cgit.freedesktop.org

  • Share