[FFmpeg-cvslog] r19744 - trunk/libavcodec/rv34.c
Reimar Döffinger
Reimar.Doeffinger
Sun Aug 30 16:43:25 CEST 2009
On Sun, Aug 30, 2009 at 07:35:37AM -0700, Mike Melanson wrote:
> Reimar D?ffinger wrote:
> > On Sun, Aug 30, 2009 at 09:47:10AM +0200, Reimar D?ffinger wrote:
> >> On Sun, Aug 30, 2009 at 08:30:53AM +0200, kostya wrote:
> >>> @@ -37,6 +37,10 @@
> >>>
> >>> //#define DEBUG
> >>>
> >>> +#define ZERO8x2(dst, stride) \
> >>> + fill_rectangle(dst, 1, 2, stride, 0, 4); \
> >>> + fill_rectangle(((uint8_t*)(dst))+4, 1, 2, stride, 0, 4); \
> >>> +
> >> Please, make this a static inline function
> >>
> >>> - if(block_type == RV34_MB_B_BACKWARD || block_type == RV34_MB_B_FORWARD)
> >>> - fill_rectangle(cur_pic->motion_val[!dir][mv_pos], 2, 2, s->b8_stride, 0, 4);
> >>> + if(block_type == RV34_MB_B_BACKWARD || block_type == RV34_MB_B_FORWARD){
> >>> + ZERO8x2(cur_pic->motion_val[!dir][mv_pos], s->b8_stride);
> >>> + }
> >> Instead of doing such confusing hacks that sooner or later someone will
> >> break due to misunderstanding (no, such a break probably will not hit
> >> SVN but still it is unkind to set up such traps in code).
> >
> > Uh, and of course thanks for the fixes. I'm about to check if that
> > fixed all issues (unfortunately our FATE Sparc/Solaris machine seems to
> > be not working since over a week...).
>
> Those belong to Jeff Downs.
>
> Jeff?
And you should update the mtv spec, the change was only a commit that
made FFmpeg flip the image, it looks far better when people aren't
walking on the ceiling anymore...
More information about the ffmpeg-cvslog
mailing list