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