D6542: Fixes to GrepOutputView toolbar

Kevin Funk noreply at phabricator.kde.org
Sat Jul 8 23:06:25 UTC 2017


kfunk accepted this revision.
kfunk added a comment.
This revision is now accepted and ready to land.


  Will push

INLINE COMMENTS

> grepoutputview.cpp:407
>  {
> -    m_prev->setEnabled(model()->rowCount());
> -    m_next->setEnabled(model()->rowCount());
> +    updateButtonState(model()->rowCount());
>  }

`model()->rowCount() > 0` to avoid implicit int -> bool conversion

REPOSITORY
  R33 KDevPlatform

BRANCH
  grep

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

To: croick, #kdevelop, kfunk
Cc: kfunk, kdevelop-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20170708/17f423b3/attachment.html>


More information about the KDevelop-devel mailing list