[Commits] [wesnoth/wesnoth] b65080: Fix utf8::lowercase (bug #22139)

GitHub noreply at github.com
Wed Jun 4 09:27:38 UTC 2014


  Branch: refs/heads/master
  Home:   https://github.com/wesnoth/wesnoth
  Commit: b6508091a23c92b6752e9f8e5257aaa58cdc2cde
      https://github.com/wesnoth/wesnoth/commit/b6508091a23c92b6752e9f8e5257aaa58cdc2cde
  Author: Chris Beck <render787 at gmail.com>
  Date:   2014-06-04 (Wed, 04 Jun 2014)

  Changed paths:
    M src/serialization/unicode.cpp
    M src/tests/test_util.cpp

  Log Message:
  -----------
  Fix utf8::lowercase (bug #22139)

As ucs4::char is unsigned, we don't need to compare against the min value of
the type. Worse, we cast the (possibly signed) min value to ucs4::char, which
on plenty of systems, results in 2^31, excluding all valid unicode codepoints.

Fix by gfgtdf.
Patch by iceiceice. (PR #189)
Commit message by ai0867.





More information about the Commits mailing list