[Ffmpeg-cvslog] r7112 - trunk/libavcodec/vc1dsp.c

Kostya kostya.shishkov
Sat Nov 18 06:11:10 CET 2006


On Fri, Nov 17, 2006 at 08:32:43AM +0100, Guillaume POIRIER wrote:
> Hi,
> 
> On 11/17/06, kostya <subversion at mplayerhq.hu> wrote:
> >Author: kostya
> >Date: Fri Nov 17 07:10:52 2006
> >New Revision: 7112
> >
> >Modified:
> >   trunk/libavcodec/vc1dsp.c
> >
> >Log:
> >Fix comments
> >
> >Modified: trunk/libavcodec/vc1dsp.c
> >==============================================================================
> >--- trunk/libavcodec/vc1dsp.c   (original)
> >+++ trunk/libavcodec/vc1dsp.c   Fri Nov 17 07:10:52 2006
> >@@ -29,7 +29,7 @@
> > #include "dsputil.h"
> >
> >
> >-/** Apply overlap transform to vertical edge
> >+/** Apply overlap transform to horizontal edge
> > */
> > static void vc1_v_overlap_c(uint8_t* src, int stride, int rnd)
> > {
> >@@ -49,7 +49,7 @@
> >     }
> > }
> >
> >-/** Apply overlap transform to horizontal edge
> >+/** Apply overlap transform to vertical edge
> > */
> > static void vc1_h_overlap_c(uint8_t* src, int stride, int rnd)
> > {
> 
> Does this call for a rename of vc1_v_overlap_c to vc1_h_overlap_c and
> vice versa?

Not necessarily. vc1_v_overlap() filters horizontal edge but in vertical
direction, so both names suits it while comments were plain wrong.

> 
> Guillaume
> -- 
> With DADVSI (http://en.wikipedia.org/wiki/DADVSI), France finally has
> a lead on USA on selling out individuals right to corporations!
> Vive la France!
> 




More information about the ffmpeg-cvslog mailing list