> 在 2017年12月15日，上午12:29，Jeyapal, Karthick <[email protected]> 写道： > >> On 12/14/17, 8:24 PM, "Steven Liu" <[email protected]> wrote: >> >> >>> 在 2017年12月14日，下午6:55，[email protected] 写道： >>> […] >>> +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 > [email protected] > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel _______________________________________________ ffmpeg-devel mailing list [email protected] http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
