Review Request: MusicBrainz and MusicDNS services support implementation

Sergey Ivanov 123kash at gmail.com
Fri Sep 24 15:00:36 CEST 2010


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

(Updated 2010-09-24 13:00:36.289289)


Review request for amarok.


Changes
-------

Rewrite It to use ffmpeg for decompression purposes.
Return missed checks to src/CMakeLists.txt.


Summary
-------

MusicBrainz and MusicDNS services support implementation.
There are three independent parts:
1. MusicBrainzFinder class - used to make requests to MusicBrainz server and process replies. All relies process in separate threads by MusicBrainzXmlParser class. For a search uses guessed from a file name track information.  No external dependences required.
2. MusicDNSFinder class - used for the same purpose as MusicBrainzFinder, but i communicate with musicdns server and receives track's PUID as a reply. Replies ether process in separate threads by MusicDNSXmlParser class. Fingerprints generated by libofa (the only external dependence in entire patch). For track decompressing (MusicDNSAudioDecoder class) used xine engine (I'm not sure is It a good choice, but amarok based on phonon media-engine, that uses xine. So we don't deed to pull any other dependences). Received PUIDs sends to MusicBrainzFinder class, for a search routine.
3. View. All classes used for store (MusicBrainzTagsModel, MusicBrainzTrackListModel) and display (MusicBrainzTagsModelDelegate) purposes.


Diffs (updated)
-----

  CMakeLists.txt 191b02e 
  cmake/modules/FindFFmpeg.cmake PRE-CREATION 
  cmake/modules/FindLibOFA.cmake PRE-CREATION 
  config-amarok.h.cmake 981b7b7 
  src/CMakeLists.txt 42614ff 
  src/core-impl/capabilities/timecode/TimecodeEditCapability.h 6e15303 
  src/core-impl/capabilities/timecode/TimecodeEditCapability.cpp 8205d45 
  src/core-impl/collections/mediadevicecollection/MediaDeviceMeta.h e35b57f 
  src/core-impl/collections/mediadevicecollection/MediaDeviceMeta.cpp 9be62d9 
  src/core-impl/collections/proxycollection/ProxyCollectionMeta.cpp 8bdbf75 
  src/core-impl/collections/sqlcollection/CapabilityDelegateImpl.cpp b5cb083 
  src/core-impl/collections/sqlcollection/SqlMeta.h ee3ec21 
  src/core-impl/collections/sqlcollection/SqlMeta.cpp 2da0333 
  src/core-impl/meta/file/File.h 6d4d395 
  src/core-impl/meta/file/File.cpp 30cd2ff 
  src/core-impl/meta/file/TagLibUtils.cpp 15b64a4 
  src/core-impl/meta/proxy/MetaProxy.h 2ef3805 
  src/core-impl/meta/proxy/MetaProxy.cpp 341e076 
  src/core/capabilities/EditCapability.h 79344bd 
  src/dialogs/MusicBrainzTagger.h PRE-CREATION 
  src/dialogs/MusicBrainzTagger.cpp PRE-CREATION 
  src/dialogs/MusicBrainzTagger.ui PRE-CREATION 
  src/dialogs/TagDialog.h 50cd801 
  src/dialogs/TagDialog.cpp 26d4eb8 
  src/dialogs/TagDialogBase.ui 9974d0b 
  src/musicbrainz/DecodedAudioData.h PRE-CREATION 
  src/musicbrainz/DecodedAudioData.cpp PRE-CREATION 
  src/musicbrainz/MusicBrainzFinder.h PRE-CREATION 
  src/musicbrainz/MusicBrainzFinder.cpp PRE-CREATION 
  src/musicbrainz/MusicBrainzMetaClasses.h PRE-CREATION 
  src/musicbrainz/MusicBrainzMetaClasses.cpp PRE-CREATION 
  src/musicbrainz/MusicBrainzTagsModel.h PRE-CREATION 
  src/musicbrainz/MusicBrainzTagsModel.cpp PRE-CREATION 
  src/musicbrainz/MusicBrainzTagsModelDelegate.h PRE-CREATION 
  src/musicbrainz/MusicBrainzTagsModelDelegate.cpp PRE-CREATION 
  src/musicbrainz/MusicBrainzTrackListModel.h PRE-CREATION 
  src/musicbrainz/MusicBrainzTrackListModel.cpp PRE-CREATION 
  src/musicbrainz/MusicBrainzXmlParser.h PRE-CREATION 
  src/musicbrainz/MusicBrainzXmlParser.cpp PRE-CREATION 
  src/musicbrainz/MusicDNSAudioDecoder.h PRE-CREATION 
  src/musicbrainz/MusicDNSAudioDecoder.cpp PRE-CREATION 
  src/musicbrainz/MusicDNSFinder.h PRE-CREATION 
  src/musicbrainz/MusicDNSFinder.cpp PRE-CREATION 
  src/musicbrainz/MusicDNSXmlParser.h PRE-CREATION 
  src/musicbrainz/MusicDNSXmlParser.cpp PRE-CREATION 
  tests/core-impl/collections/proxycollection/TestProxyCollectionMeta.cpp 55d1914 

Diff: http://git.reviewboard.kde.org/r/100000/diff


Testing
-------


Thanks,

Sergey

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/amarok-devel/attachments/20100924/cf0a226f/attachment-0001.htm 


More information about the Amarok-devel mailing list