Re: [FFmpeg-devel] H264 cleanup performance effects

2015-03-22 Thread wm4
On Sun, 22 Mar 2015 13:07:09 -0400 "Ronald S. Bultje" wrote: > Hi, > > On Sun, Mar 22, 2015 at 12:15 PM, wm4 wrote: > > > I hope FFmpeg doesn't value speed over clean code. (Oh who am I > > kidding, the worst abuse is ok if it makes it 1% faster!) > > > I think this is unfair. Your statement

Re: [FFmpeg-devel] H264 cleanup performance effects

2015-03-22 Thread Ronald S. Bultje
Hi, On Sun, Mar 22, 2015 at 12:15 PM, wm4 wrote: > I hope FFmpeg doesn't value speed over clean code. (Oh who am I > kidding, the worst abuse is ok if it makes it 1% faster!) I think this is unfair. Your statements suggests: - that the slowdown is a direct effect of making the code cleaner; -

Re: [FFmpeg-devel] H264 cleanup performance effects

2015-03-22 Thread wm4
On Sun, 22 Mar 2015 18:11:34 +0200 Ivan Kalvachev wrote: > On 3/22/15, wm4 wrote: > > On Sun, 22 Mar 2015 15:12:04 +0100 > > Michael Niedermayer wrote: > > > >> Hi anton, everyone else > > > > I appreciate that you contacted the author of these patches as well. > > > >> the "cosmetic" commits f

Re: [FFmpeg-devel] H264 cleanup performance effects

2015-03-22 Thread Ivan Kalvachev
On 3/22/15, wm4 wrote: > On Sun, 22 Mar 2015 15:12:04 +0100 > Michael Niedermayer wrote: > >> Hi anton, everyone else > > I appreciate that you contacted the author of these patches as well. > >> the "cosmetic" commits from yesterday >> (665e0c10a63d31dd6c7b1fba14db074625d54614..fa7c08d5e192aea77

Re: [FFmpeg-devel] H264 cleanup performance effects

2015-03-22 Thread wm4
On Sun, 22 Mar 2015 15:12:04 +0100 Michael Niedermayer wrote: > Hi anton, everyone else I appreciate that you contacted the author of these patches as well. > the "cosmetic" commits from yesterday > (665e0c10a63d31dd6c7b1fba14db074625d54614..fa7c08d5e192aea77fdfb7f52c44c196a3ba4452) > / > (d8a

[FFmpeg-devel] H264 cleanup performance effects

2015-03-22 Thread Michael Niedermayer
Hi anton, everyone else the "cosmetic" commits from yesterday (665e0c10a63d31dd6c7b1fba14db074625d54614..fa7c08d5e192aea77fdfb7f52c44c196a3ba4452) / (d8a45d2d49f54fde042b195f9d5859251252493d..c28ed1d743443e783537d279ae721be3bbdf7646) cause a ~1% speed loss in the H.264 decoder which is probably