[Log4cplus-devel] [ log4cplus-Bugs-3585440 ] Fix for std::codecvt in property.cxx

[SourceForge.net <no...@so...>]

Bugs item #3585440, was opened at 2012-11-08 10:28
Message generated for change (Settings changed) made by wilx
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=429073&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: Other
>Group: v1.1.x
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=429073&aid=3585440&group_id=40830