D9042: Add -Werror=switch and -Wcovered-switch-default
Aaron Puchert
noreply at phabricator.kde.org
Mon Dec 18 23:28:07 UTC 2017
aaronpuchert added a comment.
@mwolff Are the additional changes Ok? There are no new warnings anymore.
I used `Q_UNREACHABLE()` when the enumeration was used safely and a fallback when unchecked `static_cast`s were involved. In an ideal world, I would probably try to make sure that enumerations only ever have valid values, but right now I don't understand enough of the code base to properly fix this.
REPOSITORY
R32 KDevelop
BRANCH
switch-enum
REVISION DETAIL
https://phabricator.kde.org/D9042
To: aaronpuchert, #kdevelop, mwolff
Cc: mwolff, apol, kdevelop-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20171218/a1742793/attachment.html>
More information about the KDevelop-devel
mailing list