Remove unrequired private declarations for taglibextractor
ClosedPublic

Authored by astippich on Nov 25 2019, 10:46 AM.

Details

Summary

They do not need to be declared in the header

Diff Detail

Repository
R286 KFileMetaData
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
astippich created this revision.Nov 25 2019, 10:46 AM
Restricted Application added projects: Frameworks, Baloo. · View Herald TranscriptNov 25 2019, 10:46 AM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
astippich requested review of this revision.Nov 25 2019, 10:46 AM
ngraham accepted this revision.Nov 25 2019, 3:21 PM
This revision is now accepted and ready to land.Nov 25 2019, 3:21 PM
bruns requested changes to this revision.Nov 26 2019, 11:39 PM
bruns added inline comments.
src/extractors/taglibextractor.cpp
97

This puts readGenericProperties(...) with external linkage in the global namespace.

Put the private helpers in an anonymous namespace (and also include extractAudioProperties(...) in it).

This revision now requires changes to proceed.Nov 26 2019, 11:39 PM
astippich added inline comments.Nov 27 2019, 8:12 AM
src/extractors/taglibextractor.cpp
97

Urgh, that was the plan actually. I could swear I've done that.

astippich updated this revision to Diff 70398.Nov 27 2019, 8:20 AM
  • put all helper functions into anonymous namespace
bruns accepted this revision.Dec 12 2019, 8:14 PM
This revision is now accepted and ready to land.Dec 12 2019, 8:14 PM
This revision was automatically updated to reflect the committed changes.