avcodec/avpacket: deprecate av_copy_packet_side_data()
authorJames Almer <jamrial@gmail.com>
Mon, 25 Sep 2017 20:17:12 +0000 (17:17 -0300)
committerJames Almer <jamrial@gmail.com>
Sun, 1 Oct 2017 22:19:22 +0000 (19:19 -0300)
It leaks memory and destroys the dst packet in case of failure, and it
ultimately duplicates functionality already existing in the saner
av_packet_copy_props().

Reviewed-by: wm4
Signed-off-by: James Almer <jamrial@gmail.com>
libavcodec/avcodec.h

index dd7c4b4..52cc5b0 100644 (file)
@@ -4632,7 +4632,10 @@ int av_copy_packet(AVPacket *dst, const AVPacket *src);
  * Copy packet side data
  *
  * @return 0 on success, negative AVERROR on fail
+ *
+ * @deprecated Use av_packet_copy_props
  */
+attribute_deprecated
 int av_copy_packet_side_data(AVPacket *dst, const AVPacket *src);
 
 /**