Differential D24478

[dolphin/search] Add method isIndexEnabled()
ClosedPublic
Actions

Authored by iasensio on Oct 7 2019, 8:31 PM.

Details

Reviewers
elvisangelaccio
ngraham
meven
Commits
R318:df9ca5b0c223: [dolphin/search] Add method isIndexEnabled()
Summary

Extracts the logic for checking if the current path is indexed to its own method.

Test Plan

No behavior changes

Diff Detail

Repository
R318 Dolphin
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
iasensio created this revision.Oct 7 2019, 8:31 PM
Restricted Application added a project: Dolphin. · View Herald TranscriptOct 7 2019, 8:31 PM
Restricted Application added a subscriber: kfm-devel. · View Herald Transcript
iasensio requested review of this revision.Oct 7 2019, 8:31 PM
ngraham accepted this revision.Oct 7 2019, 8:36 PM
This revision is now accepted and ready to land.Oct 7 2019, 8:36 PM
meven accepted this revision.Oct 8 2019, 7:31 AM
meven added a subscriber: meven.

Nice code improvement

Closed by commit R318:df9ca5b0c223: [dolphin/search] Add method isIndexEnabled() (authored by iasensio, committed by elvisangelaccio). · Explain WhyOct 8 2019, 7:45 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 67519

View Options

src/search/dolphinsearchbox.h

Loading...
View Options

src/search/dolphinsearchbox.cpp

Loading...