Review Request: Remove setUidUrl function from EditCapability

Ralf Engels ralf-engels at gmx.de
Sun Jan 15 23:01:57 UTC 2012


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

Review request for Amarok.


Description
-------

Remove setUidUrl function from EditCapability.
The only reference to that was in the Edit dialog, but the code was unreachable.

Also remove setUidUrl from different classes where it was not used.
Conceptually it should not be possible for the unique id of a track to change as we are using it as identifier.
Different uid means different track.

50 code lines less.


Diffs
-----

  src/core-impl/capabilities/timecode/TimecodeEditCapability.h 9c6d553 
  src/core-impl/capabilities/timecode/TimecodeEditCapability.cpp baeed46 
  src/core-impl/collections/mediadevicecollection/MediaDeviceEditCapability.h 7acfb2b 
  src/core-impl/collections/mediadevicecollection/MediaDeviceEditCapability.cpp 992c8a5 
  src/core-impl/collections/mediadevicecollection/MediaDeviceMeta.h b44c8eb 
  src/core-impl/collections/mediadevicecollection/MediaDeviceMeta.cpp f4d9af4 
  src/core-impl/collections/proxycollection/ProxyCollectionMeta.cpp 05411a3 
  src/core-impl/meta/file/File.h 9e02547 
  src/core-impl/meta/file/File.cpp 9832c6f 
  src/core-impl/meta/proxy/MetaProxy.h 1893f4b 
  src/core-impl/meta/proxy/MetaProxy.cpp 3c33624 
  src/core/capabilities/EditCapability.h 8064e6a 
  src/dialogs/TagDialog.cpp a2ee45d 

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


Testing
-------

All test cases executed.
Compiled and running.


Thanks,

Ralf Engels

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


More information about the Amarok-devel mailing list