Review Request 129920: Replace Deprecated AV Functions
Dominik Waurenschk
dominik.waurenschk at gmail.com
Sat Feb 4 21:48:26 UTC 2017
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129920/
-----------------------------------------------------------
Review request for Amarok.
Repository: amarok
Description
-------
Replace all deprecated (and not found) functions with their new counterpart.
At least `avcodec_alloc_frame()` and `avcodec_get_frame_defaults()` must be replaced with `av_frame_alloc()` and `av_frame_unref()` respectively as those cause the compile to crash otherwise.
`av_free_packet()` should also be replaced with `av_packet_unref()`
Replacing how pCodecCdx is obtained should also be fine.
My only concerns are with replacing `avcodec_decode_audio4()` as this has become quite more complicated and I cannot currently test its functionality as getting tags from musicbrainz fails before it reaches this code. See [bug 328359](https://bugs.kde.org/show_bug.cgi?id=328359)
Diffs
-----
src/musicbrainz/MusicDNSAudioDecoder.cpp ea39a1b4fc
Diff: https://git.reviewboard.kde.org/r/129920/diff/
Testing
-------
Thanks,
Dominik Waurenschk
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/amarok-devel/attachments/20170204/3708c53c/attachment.html>
More information about the Amarok-devel
mailing list