D9110: Cache value before loop

David Faure noreply at phabricator.kde.org
Sat Dec 2 17:30:35 UTC 2017


dfaure requested changes to this revision.
dfaure added a comment.
This revision now requires changes to proceed.


  Good idea.

INLINE COMMENTS

> backgroundchecker.cpp:59
>          sentenceOffset=-1;
> +        bool autodetectLanguage = currentDict.testAttribute(Speller::AutoDetectLanguage);
> +        bool ignoreUpperCase = !currentDict.testAttribute(Speller::CheckUppercase);

make those bools const, for readability.

REPOSITORY
  R246 Sonnet

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

To: mlaurent, dfaure, sandsmark
Cc: #frameworks
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20171202/3d0619e8/attachment.html>


More information about the Kde-frameworks-devel mailing list