On Tue, Sep 04, 2012 at 04:20:19PM +0300, Martin Storsjö wrote:
> From: Michael Niedermayer <[email protected]>
> 
> It is possible in various error pathes as well as gap handling

paths

> that this has already been allocated. Its not clear why that

It is

> would be a problem with the current code, thus disable the
> assert to avoid common assert failure when asserts are enabled.

a common

> --- a/libavcodec/h264.c
> +++ b/libavcodec/h264.c
> @@ -2773,7 +2773,7 @@ static int decode_slice_header(H264Context *h, 
> H264Context *h0)
>              }
>          } else {
>              /* Frame or first field in a potentially complementary pair */
> -            assert(!s0->current_picture_ptr);
> +//             assert(!s0->current_picture_ptr);
>              s0->first_field = FIELD_PICTURE;
>          }

I'd drop the code instead of leaving cruft behind.

Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to