Extract front cover in the taglib extractor
AbandonedPublic

Authored by astippich on Nov 25 2019, 11:03 AM.

Details

Reviewers
bruns
mgallien
ngraham
Group Reviewers
Baloo
Summary

Add the possibility to extract the front cover
image as binary data when the corresponding flag
is set.

Diff Detail

Repository
R286 KFileMetaData
Branch
taglibBinaryExtraction
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 23903
Build 23921: arc lint + arc unit
astippich created this revision.Nov 25 2019, 11:03 AM
Restricted Application added projects: Frameworks, Baloo. · View Herald TranscriptNov 25 2019, 11:03 AM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
astippich requested review of this revision.Nov 25 2019, 11:03 AM

This is probably how it should have been implemented from the beginning, and not how I did it in the embeddedimagedata class, which duplicates a lot of code and requires additional file operations.
Baloo can continue to work as is due to the new flags.
Taglib writer will follow if this is accepted. Afterwards, I will deprecate embeddedimagedata

ngraham accepted this revision.Nov 25 2019, 11:13 PM
This revision is now accepted and ready to land.Nov 25 2019, 11:13 PM
astippich updated this revision to Diff 73098.Jan 8 2020, 8:29 PM
  • adapt to changes
astippich updated this revision to Diff 77956.Mar 18 2020, 6:44 PM
  • adapt to changes
astippich abandoned this revision.Apr 19 2020, 6:51 PM