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