On 10/06/2018 16:56, Roger Leigh wrote:
Firstly, I think you have found a bug with a combination of
configuration options we haven't explicitly tested with VS2017, namely:
- icu
- char16_t as XMLCh
ICU is expecting wchar_t, and we need to explicitly reinterpret_cast
from e.g. const char16_t * to const wchar_t *.
I thought we had already fixed all these cases, but there are so many
different ways you can configure Xerces-C++, it's quite possible I've
missed this particular combination.
Actually, just to follow up, new versions of ICU switched to char16_t a
few releases back. I think if you used a newer version of ICU, you
would find this would work seamlessly.
However, we probably want to add appropriate casts to the ICU type so
that it works with all ICU versions.
Regards,
Roger
