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