Differential D16263

[Scheduler] Remove superfluous m_extractorIdle variable
ClosedPublic
Actions

Authored by bruns on Oct 16 2018, 9:54 PM.

Details

Reviewers
poboiko
ngraham
Group Reviewers
Baloo
Frameworks
Commits
R293:a8f03d1a374a: [Scheduler] Remove superfluous m_extractorIdle variable
Summary

Having the variable is quite pointless, as hardly anyone runs baloo
with compiled with Q_ASSERT enabled.

Test Plan

make
no functional change

Diff Detail

Repository
R293 Baloo
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
bruns created this revision.Oct 16 2018, 9:54 PM
Restricted Application added projects: Frameworks, Baloo. · View Herald TranscriptOct 16 2018, 9:54 PM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
bruns requested review of this revision.Oct 16 2018, 9:54 PM
bruns updated this revision to Diff 43767.Oct 16 2018, 10:27 PM

add missing line to patch

ngraham accepted this revision.Oct 17 2018, 1:41 AM
This revision is now accepted and ready to land.Oct 17 2018, 1:41 AM
Closed by commit R293:a8f03d1a374a: [Scheduler] Remove superfluous m_extractorIdle variable (authored by bruns). · Explain WhyOct 17 2018, 1:47 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 43771

View Options

src/file/extractorprocess.h

Loading...
View Options

src/file/extractorprocess.cpp

Loading...