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