[Differential] [Commented On] D1535: add some tests for corrupt configs

[sebas (Sebastian Kügler)]

sebas added a comment.


  I found the logic puzzling at first as well, but what the Serializer does it it takes a config object and tries to change it. (Not my idea, but it seems to work quite OK overall). Serializer::config() is a bit special then because it can do two things: return an invalid pointer or not change the config. This logic is not something I want to change, we have enough bugs in the code as-is, and the last thing I want to do is destabilize it, so I'd rather have it do some additional checking.

REPOSITORY
  rKSCREEN KScreen

BRANCH
  sebas/corruptautotests

REVISION DETAIL
  https://phabricator.kde.org/D1535

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: sebas, Plasma, graesslin
Cc: broulik, plasma-devel, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160504/019d60b5/attachment.html>