diff options
| author | David Ostrovsky <david@ostrovsky.org> | 2016-03-13 09:18:00 +0100 |
|---|---|---|
| committer | Stephan Bergmann <sbergman@redhat.com> | 2016-05-03 19:26:06 +0000 |
| commit | e16fa715c43dcdf836ce8c400b6d54eae87b627d (patch) | |
| tree | 356eb41a6333db58a318a2b5da31a9c0893e90a8 /solenv | |
| parent | a2aea8bac55cef23297573733ba28f563f5aa791 (diff) | |
Handle wchar_t as native C++11 type on windows
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
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Diffstat (limited to 'solenv')
| -rw-r--r-- | solenv/gbuild/platform/com_MSC_defs.mk | 2 | ||||
| -rw-r--r-- | solenv/gcc-wrappers/wrapper.cxx | 1 |
2 files changed, 0 insertions, 3 deletions
diff --git a/solenv/gbuild/platform/com_MSC_defs.mk b/solenv/gbuild/platform/com_MSC_defs.mk index 951d859902a4..a4c801f85f6c 100644 --- a/solenv/gbuild/platform/com_MSC_defs.mk +++ b/solenv/gbuild/platform/com_MSC_defs.mk @@ -166,7 +166,6 @@ gb_CFLAGS := \ -wd4626 \ -wd4706 \ -wd4800 \ - -Zc:wchar_t- \ ifeq ($(COM_IS_CLANG),TRUE) gb_CFLAGS += \ @@ -214,7 +213,6 @@ gb_CXXFLAGS := \ -wd4706 \ -wd4800 \ -wd4913 \ - -Zc:wchar_t- \ ifeq ($(CPUNAME),X86_64) diff --git a/solenv/gcc-wrappers/wrapper.cxx b/solenv/gcc-wrappers/wrapper.cxx index 0ed323c3af5e..cc57698a3aa9 100644 --- a/solenv/gcc-wrappers/wrapper.cxx +++ b/solenv/gcc-wrappers/wrapper.cxx @@ -89,7 +89,6 @@ string processccargs(vector<string> rawargs) { else args.append(" -MD"); args.append(" -Gy"); - args.append(" -Zc:wchar_t-"); args.append(" -Ob1 -Oxs -Oy-"); // apparently these must be at the end |