Differential D17459 Diff 47907 src/search/katesearchbar.h

Changeset View

Standalone View

View Options

src/search/katesearchbar.h

Show First 20 Lines • Show All 91 Lines • ▼ Show 20 Line(s)95public Q_SLOTS:
104 * @param replacementPattern the replacement pattern 104​ * @param replacementPattern the replacement pattern
105 */ 105​ */
106 void setReplacementPattern(const QString &replacementPattern); 106​ void setReplacementPattern(const QString &replacementPattern);
107 107​
108 void setSearchMode(SearchMode mode); 108​ void setSearchMode(SearchMode mode);
109 void setSelectionOnly(bool selectionOnly); 109​ void setSelectionOnly(bool selectionOnly);
110 void setMatchCase(bool matchCase); 110​ void setMatchCase(bool matchCase);
111 111​
112 // Called for <F3> and <Shift>+<F3> 112​ // Called by buttons and typically <F3>/<Shift>+<F3> shortcuts
113 void findNext(); 113​ void findNext();
114 void findPrevious(); 114​ void findPrevious();
115 void findAll(); ​
116 115​
116​ // PowerMode stuff
117​ void findAll();
117 void replaceNext(); 118​ void replaceNext();
118 void replaceAll(); 119​ void replaceAll();
119 120​
120 // Also used by KTextEditor::ViewPrivate 121​ // Also used by KTextEditor::ViewPrivate
121 void enterPowerMode(); 122​ void enterPowerMode();
122 void enterIncrementalMode(); 123​ void enterIncrementalMode();
123 124​
124 bool clearHighlights(); 125​ bool clearHighlights();
Show All 10 Lines135private Q_SLOTS:
135 void onIncPatternChanged(const QString &pattern); 136​ void onIncPatternChanged(const QString &pattern);
136 void onMatchCaseToggled(bool matchCase); 137​ void onMatchCaseToggled(bool matchCase);
137 138​
138 void onReturnPressed(); 139​ void onReturnPressed();
139 void updateSelectionOnly(); 140​ void updateSelectionOnly();
140 void updateIncInitCursor(); 141​ void updateIncInitCursor();
141 142​
142 void onPowerPatternChanged(const QString &pattern); 143​ void onPowerPatternChanged(const QString &pattern);
143 ​
144 void onPowerModeChanged(int index); 144​ void onPowerModeChanged(int index);
145 void onPowerPatternContextMenuRequest(); 145​ void onPowerPatternContextMenuRequest();
146 void onPowerPatternContextMenuRequest(const QPoint &); 146​ void onPowerPatternContextMenuRequest(const QPoint &);
147 void onPowerReplacmentContextMenuRequest(); 147​ void onPowerReplacmentContextMenuRequest();
148 void onPowerReplacmentContextMenuRequest(const QPoint &); 148​ void onPowerReplacmentContextMenuRequest(const QPoint &);
149​ void onPowerCancelFindOrReplace();
150​
151​ /**
152​ * This function do the hard search & replace work in time slice steps.
153​ * When all is done @ref m_matchCounter is set and the signal
154​ * @ref findOrReplaceAllFinished() is emitted.
155​ */
156​ void findOrReplaceAll();
157​
158​
159​ /**
160​ * Restore needed settings when signal @ref findOrReplaceAllFinished()
161​ * was received.
162​ */
163​ void endFindOrReplaceAll();
164​
165​Q_SIGNALS:
166​ /**
167​ * Will emitted by @ref findOrReplaceAll() when all is done.
168​ */
169​ void findOrReplaceAllFinished();
149 170​
150private: 171​private:
151 // Helpers 172​ // Helpers
152 bool find(SearchDirection searchDirection = SearchForward, const QString *replacement = nullptr); 173​ bool find(SearchDirection searchDirection = SearchForward, const QString *replacement = nullptr); //BCI: Remove
153 int findAll(KTextEditor::Range inputRange, const QString *replacement); 174​ //bool find(SearchDirection searchDirection = SearchForward) { return findOrReplace(searchDirection, nullptr); }; //BCI: Enable
anthonyfieroniUnsubmitted
Not Done

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

anthonyfieroni: It's exported class you cannot remove a function, it breaks the ABI
175​ int findAll(KTextEditor::Range inputRange, const QString *replacement); //BCI: Remove
176​ bool findOrReplace(SearchDirection searchDirection, const QString *replacement);
177​
178​ /**
179​ * The entry point to start a search & replace task.
180​ * Set needed member variables and call @ref findOrReplaceAll() to do the work.
181​ */
182​ void beginFindOrReplaceAll(KTextEditor::Range inputRange, const QString &replacement, bool replaceMode = true);
183​ void beginFindAll(KTextEditor::Range inputRange) { beginFindOrReplaceAll(inputRange, QString(), false); };
anthonyfieroniUnsubmitted
Not Done

Do not change.

anthonyfieroni: Do not change.
154 184​
155 bool isPatternValid() const; 185​ bool isPatternValid() const;
156 186​
157 KTextEditor::SearchOptions searchOptions(SearchDirection searchDirection = SearchForward) const; 187​ KTextEditor::SearchOptions searchOptions(SearchDirection searchDirection = SearchForward) const;
158 188​
159 void highlightMatch(const KTextEditor::Range &range); 189​ void highlightMatch(const KTextEditor::Range &range);
160 void highlightReplacement(const KTextEditor::Range &range); 190​ void highlightReplacement(const KTextEditor::Range &range);
161 void indicateMatch(MatchResult matchResult); 191​ void indicateMatch(MatchResult matchResult);
162 static void selectRange(KTextEditor::ViewPrivate *view, const KTextEditor::Range &range); 192​ static void selectRange(KTextEditor::ViewPrivate *view, const KTextEditor::Range &range);
163 void selectRange2(const KTextEditor::Range &range); 193​ void selectRange2(const KTextEditor::Range &range);
164 194​
165 QVector<QString> getCapturePatterns(const QString &pattern) const; 195​ QVector<QString> getCapturePatterns(const QString &pattern) const;
166 void showExtendedContextMenu(bool forPattern, const QPoint &pos); 196​ void showExtendedContextMenu(bool forPattern, const QPoint &pos);
167 197​
168 void givePatternFeedback(); 198​ void givePatternFeedback();
169 void addCurrentTextToHistory(QComboBox *combo); 199​ void addCurrentTextToHistory(QComboBox *combo);
170 void backupConfig(bool ofPower); 200​ void backupConfig(bool ofPower);
171 void sendConfig(); 201​ void sendConfig();
172 void fixForSingleLine(KTextEditor::Range &range, SearchDirection searchDirection); 202​ void fixForSingleLine(KTextEditor::Range &range, SearchDirection searchDirection);
173 203​
174 void showInfoMessage(const QString &text); 204​ void showInfoMessage(const QString &text); //BCI: Remove
anthonyfieroniUnsubmitted
Not Done

Same here.

anthonyfieroni: Same here.
205​ void showResultMessage();
175 206​
176private: 207​private:
177 KTextEditor::ViewPrivate *const m_view; 208​ KTextEditor::ViewPrivate *const m_view;
178 KateViewConfig *const m_config; 209​ KateViewConfig *const m_config;
179 QList<KTextEditor::MovingRange *> m_hlRanges; 210​ QList<KTextEditor::MovingRange *> m_hlRanges;
180 QPointer<KTextEditor::Message> m_infoMessage; 211​ QPointer<KTextEditor::Message> m_infoMessage;
181 QPointer<KTextEditor::Message> m_wrappedMessage; 212​ QPointer<KTextEditor::Message> m_wrappedMessage;
182 SearchDirection m_lastSearchDirection = SearchForward; 213​ SearchDirection m_lastSearchDirection = SearchForward;
183 214​
184 // Shared by both dialogs 215​ // Shared by both dialogs
185 QVBoxLayout *const m_layout; 216​ QVBoxLayout *const m_layout;
186 QWidget *m_widget; 217​ QWidget *m_widget;
187 218​
188 // Incremental search related 219​ // Incremental search related
189 Ui::IncrementalSearchBar *m_incUi; 220​ Ui::IncrementalSearchBar *m_incUi;
190 KTextEditor::Cursor m_incInitCursor; 221​ KTextEditor::Cursor m_incInitCursor;
191 222​
192 // Power search related 223​ // Power search related
193 Ui::PowerSearchBar *m_powerUi; 224​ Ui::PowerSearchBar *m_powerUi = nullptr;
194 225​
195 // attribute to highlight matches with 226​ // attribute to highlight matches with
196 KTextEditor::Attribute::Ptr highlightMatchAttribute; 227​ KTextEditor::Attribute::Ptr highlightMatchAttribute;
197 KTextEditor::Attribute::Ptr highlightReplacementAttribute; 228​ KTextEditor::Attribute::Ptr highlightReplacementAttribute;
198 229​
199 // Status backup 230​ // Status backup
200 bool m_incHighlightAll : 1; 231​ bool m_incHighlightAll : 1;
anthonyfieroniUnsubmitted
Not Done

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.

anthonyfieroni: You cannot add new members as well, they change object size. Since this class not use pimpl…
cullmannUnsubmitted
Not Done

This is a purely internal class.
It is only exported for the unit tests, no header files are installed, therefore any change of the layout is ok ;=)

cullmann: This is a purely internal class. It is only exported for the unit tests, no header files are…
dhaumannUnsubmitted
Not Done

Yes: as rule of thumb: only classes in the KTextEditor namespace are public API. Everything else can be freely changed. In this case, as Christoph points out, this is purely internal API.

PS: maybe we should have a KTEXTEDITOR_TEST_EXPORT macro to make this explicit. :-)

dhaumann: Yes: as rule of thumb: only classes in the KTextEditor namespace are public API. Everything…
201 bool m_incFromCursor : 1; 232​ bool m_incFromCursor : 1;
202 bool m_incMatchCase : 1; 233​ bool m_incMatchCase : 1;
203 bool m_powerMatchCase : 1; 234​ bool m_powerMatchCase : 1;
204 bool m_powerFromCursor : 1; 235​ bool m_powerFromCursor : 1;
205 bool m_powerHighlightAll : 1; 236​ bool m_powerHighlightAll : 1;
206 unsigned int m_powerMode : 2; 237​ unsigned int m_powerMode : 2;
207}; 238​};
208 239​
209#endif // KATE_SEARCH_BAR_H 240​#endif // KATE_SEARCH_BAR_H
210 241​
Context not available.