-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/114978/
-----------------------------------------------------------

Review request for Amarok.


Bugs: 296049
    https://bugs.kde.org/show_bug.cgi?id=296049


Repository: amarok


Description
-------

I've created a new class MediaArt in CollectionScanner which would manage and 
store art cache files.

Tasks accomplished
+ Validation of the $XDG_CACHE_HOME/media-art directory
+ Generation of the unique and a safe file name 
  (according to the naming guidelines mentioned in 
http://tinyurl.com/MediaArtStorageSpec)
+ Copying the art file to the specified cache directory

Tasks pending
# Local media art storage should be implemented when removable media is 
encountered
# Heuristics for finding media art
# Integration of MediaArt storage into Amarok


Diffs
-----

  shared/collectionscanner/MediaArt.h PRE-CREATION 
  shared/collectionscanner/MediaArt.cpp PRE-CREATION 

Diff: https://git.reviewboard.kde.org/r/114978/diff/


Testing
-------

Works as expected (with reference to the tasks accomplished)


Thanks,

Animesh  Ramesh

_______________________________________________
Amarok-devel mailing list
Amarok-devel@kde.org
https://mail.kde.org/mailman/listinfo/amarok-devel

Reply via email to