D10190: Fix resetting "View Modes" preferences to defaults

Elvis Angelaccio noreply at phabricator.kde.org
Sat Feb 10 18:21:12 GMT 2018


elvisangelaccio added inline comments.

INLINE COMMENTS

> viewsettingstab.cpp:235-236
>  
>      ViewModeSettings settings(viewMode());
> -    settings.readConfig();
>  

Makes sense and fixes the bug. But please make the `settings` variable `const`, like it was before the `readConfig()` call was added.

REPOSITORY
  R318 Dolphin

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

To: hoffmannrobert
Cc: ltoscano, ngraham, elvisangelaccio, michaelh, #dolphin, spoorun, navarromorales, isidorov, firef, andrebarros, emmanuelp
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20180210/9bf93d04/attachment.htm>


More information about the kfm-devel mailing list