Patches item #3585440, was opened at 2012-11-08 10:28
Message generated for change (Comment added) made by wilx
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=429075&aid=3585440&group_id=40830
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: v1.1.0
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Chris Steenwyk (csteenwyk)
Assigned to: Václav Zeman (wilx)
Summary: Fix for std::codecvt in property.cxx
Initial Comment:
std::codecvt was given an incorrect set of template parameters
----------------------------------------------------------------------
>Comment By: Václav Zeman (wilx)
Date: 2012-11-09 06:28
Message:
I have just re-tested with VS 2010. It appears that some for of "null"
codecvt that just copies the bytes into the wchar_ts is necessary.
----------------------------------------------------------------------
Comment By: Václav Zeman (wilx)
Date: 2012-11-09 06:12
Message:
I do not believe that the original unpatched line of code is entirely
incorrect. The idea behind it is that when both type parameters are
wchar_t, the compiler picks the default std::covdecvt<> template and does
no conversion over the wchar_ts. Does this not work at all for you?
Maybe it should instead be a custom "null" facet intherited from
std::codecvt<wchar_t, char> just doing memcpy()?
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=429075&aid=3585440&group_id=40830
|
