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

(Updated July 23, 2016, 1:53 a.m.)


Status
------

This change has been marked as submitted.


Review request for KDE Frameworks and Daniel Laidig.


Changes
-------

Submitted with commit 7baa78ca83ab3c47d2f03259959ec25e59b2f711 by Christoph 
Feck to branch master.


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


Repository: kwidgetsaddons


Description
-------

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 
  src/kcharselect.cpp 5649200 
  src/kcharselect_p.h 07b4ead 
  src/kcharselectdata.cpp aea958c 
  src/kcharselectdata_p.h 2f9332d 

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


Testing
-------

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

_______________________________________________
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Reply via email to