D17459: SearchBar: Add Cancel button to stop long running tasks

Anthony Fieroni noreply at phabricator.kde.org
Thu Dec 20 06:50:52 GMT 2018


anthonyfieroni added inline comments.

INLINE COMMENTS

> katesearchbar.h:153
> -    bool find(SearchDirection searchDirection = SearchForward, const QString *replacement = nullptr);
> -    int findAll(KTextEditor::Range inputRange, const QString *replacement);
>  

It's exported class you cannot remove a function, it breaks the ABI

> katesearchbar.h:172
>      // Helpers
> -    bool find(SearchDirection searchDirection = SearchForward, const QString *replacement = nullptr);
> -    int findAll(KTextEditor::Range inputRange, const QString *replacement);
> +    bool find(SearchDirection searchDirection = SearchForward) { return findOrReplace(searchDirection, nullptr); };
> +    bool findOrReplace(SearchDirection searchDirection, const QString *replacement);

Do not change.

> katesearchbar.h:174
>  
> -    void showInfoMessage(const QString &text);
>  

Same here.

> katesearchbar.h:221-227
> +    KTextEditor::MovingRange *m_workingRange = nullptr;
> +    KTextEditor::Range m_inputRange;
> +    QString m_replacement;
> +    uint m_matchCounter = 0;
> +    bool m_replaceMode = false;
> +    bool m_cancelFindOrReplace = false;
> +    std::vector<KTextEditor::Range> m_highlightRanges;

You cannot add new members as well, they change object size. Since this class not use pimpl idiom it will be harder to change anything in header except to add new non-virtual functions.

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

To: loh.tar, #ktexteditor, #vdg, cullmann
Cc: anthonyfieroni, brauch, cullmann, abetts, kwrite-devel, kde-frameworks-devel, #ktexteditor, hase, michaelh, ngraham, bruns, demsking, sars, dhaumann
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20181220/c72176c6/attachment.html>


More information about the Kde-frameworks-devel mailing list