[Tellico-users] Review Request 120487: Switch to libdiscid for the CDROM track reading

Pino Toscano pino at kde.org
Wed Oct 8 07:31:48 UTC 2014

This is an automatically generated e-mail. To reply, visit:

(Updated Ott. 8, 2014, 7:31 a.m.)

Review request for Tellico.


Added sector count as last item.

Repository: tellico


Look for and use libdiscid for reading the CDROM tracks, removing all the OS-dependent code for that.

Diffs (updated)

  CMakeLists.txt c4303e7e9274290ac32d8b75b650cf5e31ddc0ca 
  cmake/modules/FindDiscID.cmake PRE-CREATION 
  src/translators/CMakeLists.txt 807f22be68c66e455c8e6bcd246d77c958e47725 
  src/translators/freedb_util.cpp 58d34f2152f550532c41f1e74e0d84f23c993af9 
  config.h.cmake e55df834cc6210f4e8eb234fc2c53e87075cdd28 

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


Builds fine with and without libdiscid.

I don't have audio CDs around, so I cannot properly test it works as well...


Pino Toscano

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/tellico-users/attachments/20141008/cd474530/attachment.html>

More information about the tellico-users mailing list