diff options
author | Stephan Bergmann <sbergman@redhat.com> | 2012-08-20 15:18:35 +0200 |
---|---|---|
committer | Petr Mladek <pmladek@suse.cz> | 2012-08-20 16:28:44 +0200 |
commit | d3d9ec4630a63cfc8879371752644262e4d9b4b8 (patch) | |
tree | d4c261701319fe692c280cb64bc207599f1ee6ea /scaddins | |
parent | ee93da11b3395dc47e8718a8237ee2c650fbc85e (diff) |
fdo#53252: _toupper is unreliable
<http://msdn.microsoft.com/en-us/library/45119yx3%28v=vs.80%29.aspx> "toupper,
_toupper, towupper, _toupper_l, _towupper_l: Visual Studio 2005" states: "In
order for toupper to give the expected results, __isascii and isupper must both
return nonzero." But a community comment clarifies: "This is incorrect, and
should refer to '_toupper' (the macro form), not 'toupper'." (Which makes
sense, as otherwise toupper would violate the C Standard.)
And indeed, at least for some LO built against MSVC 2008 Express on Windows 7,
_toupper('C') = '#' is broken, while toupper('C') = 'C' is good. (And the
unexpected '#' then causes all sorts of errors in the code.)
Change-Id: Iddaddcaf0cc3ffb30e55b0f410a6cfe9118accc8
Signed-off-by: Petr Mladek <pmladek@suse.cz>
Diffstat (limited to 'scaddins')
0 files changed, 0 insertions, 0 deletions