[FFmpeg-cvslog] r17280 - in trunk/libavcodec: mpeg12.c mpegvideo.c mpegvideo_xvmc.c xvmc_internal.h

[Diego Biurrun]

On Sat, Feb 14, 2009 at 11:46:57PM +0200, Ivan Kalvachev wrote:
> On 2/14/09, Michael Niedermayer <michaelni at gmx.at> wrote:
> > On Sat, Feb 14, 2009 at 10:17:07PM +0100, Diego Biurrun wrote:
> >> On Sat, Feb 14, 2009 at 09:58:37PM +0100, Michael Niedermayer wrote:
> >> > On Sat, Feb 14, 2009 at 09:41:00PM +0100, Diego Biurrun wrote:
> >> > > On Sat, Feb 14, 2009 at 09:37:15PM +0100, diego wrote:
> >> > > >
> >> > > > Log:
> >> > > > Add xvmc_internal.h that contains all internal xvmc function
> >> > > > declarations.
> >> > > > This allows getting rid of a bunch of ugly forward declarations.
> >> > >
> >> > > Hey Michael, you missed this during your review, I deserve some bonus
> >> > > points here :)
> >> >
> >> > well ok, let me round it up to a full number
> >> > diego 17
> >> > ivan 4
> >>
> >> Bah, that's very few points for a good idea even you missed during
> >> review.. :-/
> >
> > yes but its not on the review that makes me feel odd if i would give more
> > for it also you alraedy have so many ...
> 
> Rules doesn't mention anything about fixing unrelated issues.
> I also did notice it, but...

What, you claim to be the maintainer and did not fix an issue you
noticed ?!?

;-p

Diego