On 2014-02-13 00:47:29 +0100, Vittorio Giovara wrote:
> ---
> Or at least they look that way.
> Any other findings would be welcome.
> Vittorio
>  libavcodec/h264.c | 1 -
>  libavcodec/h264.h | 6 +-----
>  2 files changed, 1 insertion(+), 6 deletions(-)
> 
> diff --git a/libavcodec/h264.c b/libavcodec/h264.c
> index 701a5b1..8f2a3f1 100644
> --- a/libavcodec/h264.c
> +++ b/libavcodec/h264.c
> @@ -3324,7 +3324,6 @@ int ff_set_ref_count(H264Context *h)
>  
>  /**
>   * Decode a slice header.
> - * This will also call ff_MPV_common_init() and frame_start() as needed.

maybe '* This will (re)intialize te decoder and call h264_frame_start() as 
needed.'?

undecided whether it's an useful comment or not

>   *
>   * @param h h264context
>   * @param h0 h264 master context (differs from 'h' when doing sliced based
> diff --git a/libavcodec/h264.h b/libavcodec/h264.h
> index ef81463..21b7996 100644
> --- a/libavcodec/h264.h
> +++ b/libavcodec/h264.h
> @@ -355,11 +355,7 @@ typedef struct H264Context {
>      ptrdiff_t mb_uvlinesize;
>  
>      SPS sps; ///< current sps
> -
> -    /**
> -     * current pps
> -     */
> -    PPS pps; // FIXME move to Picture perhaps? (->no) do we need that?
> +    PPS pps; ///< current pps

ok

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

Reply via email to