Re: [libav-devel] [PATCH 2/4] mpegvideo: Drop release_unused_pictures() function

2015-06-08 Thread Luca Barbato
On 08/06/15 14:00, Vittorio Giovara wrote:
 It is only used in one place, and it is small enough that it might be
 called directly.
 ---
  libavcodec/mpegvideo.c | 17 +
  1 file changed, 5 insertions(+), 12 deletions(-)
 

Ok.

___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel


[libav-devel] [PATCH 2/4] mpegvideo: Drop release_unused_pictures() function

2015-06-08 Thread Vittorio Giovara
It is only used in one place, and it is small enough that it might be
called directly.
---
 libavcodec/mpegvideo.c | 17 +
 1 file changed, 5 insertions(+), 12 deletions(-)

diff --git a/libavcodec/mpegvideo.c b/libavcodec/mpegvideo.c
index 86eab07..1ea2b65 100644
--- a/libavcodec/mpegvideo.c
+++ b/libavcodec/mpegvideo.c
@@ -1109,17 +1109,6 @@ void ff_mpv_common_end(MpegEncContext *s)
 s-linesize = s-uvlinesize = 0;
 }
 
-static void release_unused_pictures(AVCodecContext *avctx, Picture *picture)
-{
-int i;
-
-/* release non reference frames */
-for (i = 0; i  MAX_PICTURE_COUNT; i++) {
-if (!picture[i].reference)
-ff_mpeg_unref_picture(avctx, picture[i]);
-}
-}
-
 /**
  * generic function called after decoding
  * the header and before a frame is decoded.
@@ -1152,7 +1141,11 @@ int ff_mpv_frame_start(MpegEncContext *s, AVCodecContext 
*avctx)
 
 ff_mpeg_unref_picture(s-avctx, s-current_picture);
 
-release_unused_pictures(s-avctx, s-picture);
+/* release non reference frames */
+for (i = 0; i  MAX_PICTURE_COUNT; i++) {
+if (!s-picture[i].reference)
+ff_mpeg_unref_picture(s-avctx, s-picture[i]);
+}
 
 if (s-current_picture_ptr  !s-current_picture_ptr-f-buf[0]) {
 // we already have a unused image
-- 
1.9.5 (Apple Git-50.3)

___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel