D14549: EDID parsing: replace non-printable characters with '-'
Frederik Gladhorn
noreply at phabricator.kde.org
Thu Aug 2 10:25:48 BST 2018
gladhorn created this revision.
gladhorn added a reviewer: Plasma.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
gladhorn requested review of this revision.
REVISION SUMMARY
Simplify first to not have zero terminators suddenly replaced by '-'.
This makes some screen names a lot more readable since the encoding
seems always problematic. According to the standard it should be ipm
437, but even with that code page, many EDIDs seem to produce garbage.
Local8Bit also doesn't make much sense, so use Latin1 as a good
compromise and replace everything that's not in range.
REPOSITORY
R110 KScreen Library
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D14549
AFFECTED FILES
autotests/testedid.cpp
src/edid.cpp
To: gladhorn, #plasma
Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180802/7aa19676/attachment.html>
More information about the Plasma-devel
mailing list