On Mon, Mar 28, 2011 at 00:58:04 (CEST), Diego Biurrun wrote:

>> I've updated changelog and docs but don't know how to do a minor version  
>> bump.
>
> Increase LIBAVCODEC_VERSION_MINOR in libavcodec/version.h.
>
>> Also I this that this change must be done by commiter because review  
>> process can be finished after a couple of days, weeks or years.
>
> Nah, you should do it; it's easier for the committer to change your
> patch then and there are no other decoders in the queue right now,
> so there should not be conflicts.

[...]

>> --- /dev/null
>> +++ b/libavcodec/mxpegdec.c
>> @@ -0,0 +1,342 @@
>> +/*
>> + * MxPEG decoder
>> + * Copyright (c) 2011 Anatoly Nenashev
>> + *
>> + * This file is part of FFmpeg.
>> + *
>> + * FFmpeg is free software; you can redistribute it and/or
>
> This is libav, not FFmpeg, so please use the correct license header.

While both these comments are valid, I really hope that they don't
intimidate potential contributors from participating. I guess Anton
wants to submit his code for both FFmpeg and Libav, but doesn't bother
enough to maintain two versions of his patch. These issues really
shouldn't a reason to scare him off of nor to reject his patch, and I
hope this is also Anatoly's understanding!

-- 
Gruesse/greetings,
Reinhard Tartler, KeyID 945348A4
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to