On Thu, 3 Jul 2014 10:31:33 -0700, Diego Biurrun <[email protected]> wrote: > --- > > Now featuring better context member names.. > > libavcodec/Makefile | 3 +- > libavcodec/dsputil.c | 32 ------ > libavcodec/dsputil.h | 6 - > libavcodec/mpegvideo.h | 2 + > libavcodec/mpegvideo_enc.c | 13 ++- > libavcodec/mpegvideoencdsp.c | 64 +++++++++++ > libavcodec/mpegvideoencdsp.h | 41 +++++++ > libavcodec/x86/Makefile | 3 +- > libavcodec/x86/dsputilenc_mmx.c | 79 ------------- > ..._qns_template.c => mpegvideoenc_qns_template.c} | 2 +- > libavcodec/x86/mpegvideoencdsp_init.c | 125 > +++++++++++++++++++++ > 11 files changed, 245 insertions(+), 125 deletions(-) > create mode 100644 libavcodec/mpegvideoencdsp.c > create mode 100644 libavcodec/mpegvideoencdsp.h > rename libavcodec/x86/{dsputil_qns_template.c => > mpegvideoenc_qns_template.c} (98%) > create mode 100644 libavcodec/x86/mpegvideoencdsp_init.c >
Don't see the better names, but the patch looks mostly fine. > + MpegvideoEncDSPContext mvedsp; nit: IMO mpvencdsp or something along those lines would be more descriptive -- Anton Khirnov _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
