aacid created this revision.
Restricted Application added a project: Frameworks.
Restricted Application added a subscriber: kde-frameworks-devel.
aacid requested review of this revision.

REVISION SUMMARY
  Even when calling setClearButtonEnabled instead of setClearButtonShown
  
  Recently KLineEdit::setClearButtonShown got deprecated, but that meant
  that silently &KLineEdit::clearButtonClicked would not be emitted.
  
  This patch corrects it. It uses quite a bit of internal magic, but
  the old one was already using some so i think it's ok, given it even
  comes with an autotest that will break when Qt changes its internals

TEST PLAN
  Test passes :)

REPOSITORY
  R284 KCompletion

BRANCH
  master

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

AFFECTED FILES
  autotests/klineedit_unittest.cpp
  src/klineedit.cpp
  tests/klineedittest.cpp

To: aacid
Cc: davidedmundson, kde-frameworks-devel, broulik, michaelh, ngraham, bruns

Reply via email to