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
Branch
refactor_index_enabled
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 17418
Build 17436: arc lint + arc unit
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 67459

View Options

src/search/dolphinsearchbox.h

Loading...
View Options

src/search/dolphinsearchbox.cpp

Loading...