D17651: Escape bytes that are larger than 127 in config files

2018-12-18 Thread Jos van den Oever
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

D17651: Escape bytes that are larger than 127 in config files

2018-12-18 Thread David Faure
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,

D17651: Escape bytes that are larger than 127 in config files

2018-12-18 Thread Jos van den Oever
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,