vandenoever updated this revision to Diff 47788.
vandenoever edited the summary of this revision.
vandenoever added a comment.
Added a unit test and changed the code so that 0x7f (DEL) is also
escaped.
REPOSITORY
R237 KConfig
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D17651
dfaure added a comment.
+1 from me, although a unittest would be good. Especially in order to check
that this then gets decoded properly when reading back from the kconfig file...
REPOSITORY
R237 KConfig
REVISION DETAIL
https://phabricator.kde.org/D17651
To: vandenoever, dfaure, arichar
vandenoever retitled this revision from "Escape bytes that larger than 127" to
"Escape bytes that are larger than 127 in config files".
REPOSITORY
R237 KConfig
REVISION DETAIL
https://phabricator.kde.org/D17651
To: vandenoever, dfaure, arichardson
Cc: kde-frameworks-devel, michaelh, ngraham