> 在 2017年12月15日，上午12:29，Jeyapal, Karthick <kjeya...@akamai.com> 写道： > >> On 12/14/17, 8:24 PM, "Steven Liu" <l...@chinaffmpeg.org> wrote: >> >> >>> 在 2017年12月14日，下午6:55，vdi...@akamai.com 写道： >>> […] >>> +libavformat/reverse.c >> >> this need double check for a better way > The better way of doing this is to share ff_reverse function. No I don’t think that is a better way here. > But such a patch submitted recently was not pushed due to several objections. > http://ffmpeg.org/pipermail/ffmpeg-devel/2017-December/221472.html > Hence, we are left with only this option. Moreover like same approach was > used for avcodec and avdevice(to add reverse.c), as well. So, extending the > same approach for avformat shouldn’t deviate from ffmpeg’s principles. I think there have better way, Please don’t define the ffmpeg’s principles to every place. you are duplicate the code from libavcodec//reverse.c to libavformat, IMHO, that is not a good option.
Thanks Steven > > Regards, > Karthick >> >> […] >> Thanks >> >> >> Steven > > > > _______________________________________________ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
