Review Request 120904: [KCharSelect] Handle characters outside BMP

Christoph Feck cfeck at kde.org
Fri Jul 15 02:22:10 UTC 2016


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/120904/
-----------------------------------------------------------

(Updated July 15, 2016, 2:22 a.m.)


Review request for KDE Frameworks and Daniel Laidig.


Changes
-------

Implement proposed API


Summary (updated)
-----------------

[KCharSelect] Handle characters outside BMP


Bugs: 142625
    https://bugs.kde.org/show_bug.cgi?id=142625


Repository: kwidgetsaddons


Description (updated)
-------

Changed QChar to uint everywhere in KCharSelect to be able to handle non-BMP characters.

Old API is still working, unless the user does setAllPlanesEnabled(true), then it is expected that she does use the new API.

This is the first step to fix bug 142625, and should be committed as early as possible (for 5.25?), so that application developers have time to adopt the new API.


Diffs
-----

  src/kcharselect.h 05ec77c 

Diff: https://git.reviewboard.kde.org/r/120904/diff/


Testing (updated)
-------

When using utils/kcharselect without patch from https://git.reviewboard.kde.org/r/128453/ it seems to work as previously.

When applying above patch to utils/kcharselect:

* kcharselect-data still only has 16 bit data, so no blocks outside the BMP can be browsed. The code to create and parse the database still needs to be updated.
* you can search for 0x12345 unicode or 0x23456 unicode, and see some naming difference
* you can copy the resulting non-BMP character


Thanks,

Christoph Feck

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20160715/a4be4d9b/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list