From 52d336e4c1f6345b43815b26373b9068e1c464a3 Mon Sep 17 00:00:00 2001 From: wm4 Date: Fri, 11 Mar 2016 20:31:41 +0100 Subject: dec_video: reduce scope of variable Almost only a cosmetic change, although it decreases pointless referencing/dereferencing of the cover art packet too. --- video/decode/dec_video.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'video/decode') diff --git a/video/decode/dec_video.c b/video/decode/dec_video.c index fc0b090962..93a68dc468 100644 --- a/video/decode/dec_video.c +++ b/video/decode/dec_video.c @@ -363,20 +363,20 @@ void video_work(struct dec_video *d_video) return; if (d_video->header->attached_picture) { - struct demux_packet *packet = - demux_copy_packet(d_video->header->attached_picture); if (d_video->current_state == DATA_AGAIN && !d_video->cover_art_mpi) { + struct demux_packet *packet = + demux_copy_packet(d_video->header->attached_picture); d_video->cover_art_mpi = decode_packet(d_video, packet, 0); // Might need flush. if (!d_video->cover_art_mpi) d_video->cover_art_mpi = decode_packet(d_video, NULL, 0); d_video->current_state = DATA_OK; + talloc_free(packet); } if (d_video->current_state == DATA_OK) d_video->current_mpi = mp_image_new_ref(d_video->cover_art_mpi); // (DATA_OK is returned the first time, when current_mpi is sill set) d_video->current_state = DATA_EOF; - talloc_free(packet); return; } -- cgit v1.2.3