On Wed, Jun 29, 2011 at 02:13:20PM -0700, Ronald S. Bultje wrote:
>
> --- a/libavformat/matroskadec.c
> +++ b/libavformat/matroskadec.c
> @@ -1879,11 +1879,13 @@ static int
> matroska_parse_cluster(MatroskaDemuxContext *matroska)
> static int matroska_read_packet(AVFormatContext *s, AVPacket *pkt)
> {
> MatroskaDemuxContext *matroska = s->priv_data;
> + int ret;
>
> while (matroska_deliver_packet(matroska, pkt)) {
> if (matroska->done)
> return AVERROR_EOF;
> - matroska_parse_cluster(matroska);
> + ret = matroska_parse_cluster(matroska);
> + if (ret) return ret;
> }
>
> return 0;
Why not just 'return ret;'?
Diego
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel
