[FFmpeg-cvslog] r9447 - trunk/libavcodec/wmv2.c

Ivan Kalvachev ikalvachev
Thu Jun 28 15:07:10 CEST 2007


2007/6/27, Michael Niedermayer <michaelni at gmx.at>:
> Hi
>
> On Wed, Jun 27, 2007 at 04:27:48PM +0200, iive wrote:
> > Author: iive
> > Date: Wed Jun 27 16:27:47 2007
> > New Revision: 9447
> >
> > Log:
> > Identify the loop_filter flag as such and remove the unused variables in the context.
>
> you are not wmv2 maintainer, you didnt send a patch,
> you didnt provide any sample which shows artifacts without this
> and i doubt it is correct as is, wmv2 likely doesnt use the h.263
> loop filter

Even using wrong deblocker on decoding gives much better results than
not using any.

Encoding with loop_filter set to 1, as it was until now, whole not
using any filtering at all is obviously wrong. I see encoding with
enabled "loop_filter" is marked as problematic even for h263, so I
don't see it as urgent problem.
And yes,  sorry, it changes the regression, however the encoded file
should differ in only 1 bit. The decoded output should stay the same.


I'll see if the person who provided me with samples would allow me to
upload them.
I think that the 703 in samples.mphq have the loopfilter enabled,
however it have other problems (especially when playing it with win32
loader)




More information about the ffmpeg-cvslog mailing list