EDID parsing: replace non-printable characters with '-'
ClosedPublic

Authored by gladhorn on Aug 2 2018, 9:25 AM.

Details

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.

Diff Detail

Repository
R110 KScreen Library
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
gladhorn created this revision.Aug 2 2018, 9:25 AM
Restricted Application added a project: Plasma. · View Herald TranscriptAug 2 2018, 9:25 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
gladhorn requested review of this revision.Aug 2 2018, 9:25 AM
mart accepted this revision.Aug 12 2018, 2:52 PM
This revision is now accepted and ready to land.Aug 12 2018, 2:52 PM
This revision was automatically updated to reflect the committed changes.