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
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
