On Thu, Sep 22, 2016 at 04:29:09PM +0200, Anton Khirnov wrote:
> This code is independent from the decoder, so it makes more sense for it
> to to have its own header.
> ---
>  libavcodec/hevc.h        |   5 +
>  libavcodec/hevc_data.h   |  29 +++++
>  libavcodec/hevc_ps.c     |  15 +--
>  libavcodec/hevc_ps.h     | 320 
> +++++++++++++++++++++++++++++++++++++++++++++++
>  libavcodec/hevc_ps_enc.c |   2 +-
>  libavcodec/hevc_refs.c   |   2 +-
>  libavcodec/hevcdec.c     |   3 +-
>  libavcodec/hevcdec.h     | 301 +-------------------------------------------
>  8 files changed, 370 insertions(+), 307 deletions(-)
>  create mode 100644 libavcodec/hevc_data.h
>  create mode 100644 libavcodec/hevc_ps.h
> 
> --- /dev/null
> +++ b/libavcodec/hevc_data.h
> @@ -0,0 +1,29 @@
> +
> +#ifndef AVCODEC_HEVC_DATA_H
> +#define AVCODEC_HEVC_DATA_H
> +
> +extern const uint8_t ff_hevc_diag_scan4x4_x[16];
> +extern const uint8_t ff_hevc_diag_scan4x4_y[16];
> +extern const uint8_t ff_hevc_diag_scan8x8_x[64];
> +extern const uint8_t ff_hevc_diag_scan8x8_y[64];
> +
> +#endif /* AVCODEC_HEVC_DATA_H */

This should be a separate patch IMO. Also, this header will not compile
standalone. Plus, hevc_data.c was not updated to #include it.

Diego
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to