Diego Biurrun wrote:
This fixes some warnings about unused-but-set variables. --- ffplay.c | 14 ++++++-------- 1 files changed, 6 insertions(+), 8 deletions(-)diff --git a/ffplay.c b/ffplay.c index 7e61fb1..a09f207 100644 --- a/ffplay.c +++ b/ffplay.c @@ -1457,7 +1457,7 @@ static int output_picture2(VideoState *is, AVFrame *src_frame, double pts1, int6 static int get_video_frame(VideoState *is, AVFrame *frame, int64_t *pts, AVPacket *pkt) { - int len1, got_picture, i; + int got_picture, i; if (packet_queue_get(&is->videoq, pkt, 1)< 0) return -1; @@ -1485,9 +1485,8 @@ static int get_video_frame(VideoState *is, AVFrame *frame, int64_t *pts, AVPacke return 0; } - len1 = avcodec_decode_video2(is->video_st->codec, - frame,&got_picture, - pkt); + if (avcodec_decode_video2(is->video_st->codec, frame,&got_picture, pkt)< 0) + return 1;
this does not just fix a warning, it also changes behaviour, no? _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
