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

[Ivan Kalvachev]

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...