D10832: Replace all enum to enum class

Nathaniel Graham noreply at phabricator.kde.org
Mon Feb 26 13:37:00 GMT 2018


ngraham added a comment.


  In D10832#213795 <https://phabricator.kde.org/D10832#213795>, @rominf wrote:
  
  > Another variant is to apply only obvious changes and skip all changes that require `static_cast`s. It's not hard to do (I need just filter my changes), easy to check for a reviewer (verify that `(X)SomethingPostfix -> X::Something`) and better than nothing.
  
  
  +1, sounds sane.

REPOSITORY
  R318 Dolphin

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

To: rominf, #dolphin
Cc: ngraham, elvisangelaccio, markg, rkflx, #dolphin, spoorun, navarromorales, isidorov, firef, andrebarros, emmanuelp
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20180226/4148f7b0/attachment.htm>


More information about the kfm-devel mailing list