D12236: Change internal character type size to 32 bit
Mariusz Glebocki
noreply at phabricator.kde.org
Sun Apr 15 23:12:29 UTC 2018
mglb created this revision.
mglb added a reviewer: Konsole.
Restricted Application added a project: Konsole.
mglb requested review of this revision.
REVISION SUMMARY
Currently Unicode uses 0x10FFFF code points. It is possible to represent
all of them with up to two 16 bit values (UTF-16), but this makes it
harder to e.g. check for their width.
Some test cases were changed. Originally they were added with an
assumption that the code point will be truncated to 16 bit value and
as a result changed to another code point.
TEST PLAN
- All code points <= 0xFFFF should work as before
- Start the same tmux session in two Konsoles
- Change background to fully transparent in one of them and put it in front of the other one, so that all text will overlap
- Generate characters with: ``` perl -XCSDL -e 'print map{chr($_), " "} 1..0xffff' ``` (you can pipe it to `fold -s | less -r`)
- Compare output visually.
- Code points > 0xFFFF should not be truncated to 16 bits
- "𝐀" and "퐀" should be different characters
- Some code points > 0xFFFF should have single width
- Vertical lines below should align: ``` 𝐀 | 🐈| ```
REPOSITORY
R319 Konsole
BRANCH
wip/Change-internal-character-type-size-to-32-bit
REVISION DETAIL
https://phabricator.kde.org/D12236
AFFECTED FILES
src/Character.h
src/Emulation.cpp
src/Emulation.h
src/ExtendedCharTable.cpp
src/ExtendedCharTable.h
src/Screen.cpp
src/Screen.h
src/TerminalCharacterDecoder.cpp
src/TerminalDisplay.cpp
src/Vt102Emulation.cpp
src/Vt102Emulation.h
src/autotests/CharacterWidthTest.cpp
src/konsole_wcwidth.cpp
src/konsole_wcwidth.h
To: mglb, #konsole
Cc: #konsole, herrold, ngraham, hindenburg
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/konsole-devel/attachments/20180415/125ab4d4/attachment.html>
More information about the konsole-devel
mailing list