Port away from Qt 5.14 deprecated QList methods
ClosedPublic

Authored by ahmadsamir on Dec 19 2019, 11:54 AM.

Details

Summary

Refactor FileIndexerConfig::excludeFilters() as it doesn't make sense
to me to convert to a QStringList to a QSet then to QStringList, if
there is a reason I couldn't fathom it.

Test Plan

The code compiles and unit tests pass

Diff Detail

Repository
R293 Baloo
Branch
l-qt14-deprecated (branched from master)
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 20048
Build 20066: arc lint + arc unit
ahmadsamir created this revision.Dec 19 2019, 11:54 AM
Restricted Application added projects: Frameworks, Baloo. · View Herald TranscriptDec 19 2019, 11:54 AM
ahmadsamir requested review of this revision.Dec 19 2019, 11:54 AM
ahmadsamir retitled this revision from Port away from Qt 14.0 deprecated QList methods to Port away from Qt 5.14 deprecated QList methods.

Qt 14 hasn't been released yet (I kept thinking something looks wrong...)

Partially looks similar to what I did in D26069 , although that's not framework

dfaure accepted this revision.Dec 20 2019, 9:18 AM
This revision is now accepted and ready to land.Dec 20 2019, 9:18 AM
This revision was automatically updated to reflect the committed changes.

FTR, a recent discussion on #kde-devel on Freenode, which involved QSet and toSet(), made me understand why the original code was using toSet(), then fromSet(), it was to remove duplicates as QSet doesn't allow duplicates.