Review Request 109781: JJ 312407 - don't transcode from mp3 to mp3

[Anmol Ahuja]

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

(Updated March 31, 2013, 4:29 a.m.)


Review request for Amarok.


Description (updated)
-------

Added 3 checkbox in the TranscodingAssistantDialog:
"Transcode all tracks" - transcode all tracks, the current default behavior
"Ignore files which're already the selected format" - transcode only if source and setination file formats are different
"Transcode only when needed for playability" - transcode only when needed for playability in the destination collection


Diffs (updated)
-----

  src/core-impl/collections/db/sql/SqlCollectionLocation.cpp 11fa33e 
  src/core-impl/collections/ipodcollection/jobs/IpodCopyTracksJob.cpp 8a40c6c 
  src/core-impl/collections/umscollection/UmsCollectionLocation.cpp e0ba0ac 
  src/core/transcoding/TranscodingConfiguration.h 98b2bb8 
  src/core/transcoding/TranscodingConfiguration.cpp ca84f0d 
  src/transcoding/TranscodingAssistantDialog.h 76287a7 
  src/transcoding/TranscodingAssistantDialog.cpp 6bba0ec 
  src/transcoding/TranscodingAssistantDialog.ui 2505bd3 
  src/transcoding/TranscodingJob.h 6170c2a 
  src/transcoding/TranscodingJob.cpp cab76a7 
  src/transcoding/TranscodingOptionsStackedWidget.h 9495d44 
  src/transcoding/TranscodingOptionsStackedWidget.cpp cf4f328 

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


Testing (updated)
-------

Works as expected
All build tests passed


Thanks,

Anmol Ahuja

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/amarok-devel/attachments/20130330/8b1f2ad2/attachment.html>