[FFmpeg-devel] [PATCH 1/4] V14 - Adding SCTE-35 CUI codec

[Marton Balint]

On Sat, 22 Oct 2016, Kieran Kunhya wrote:

> On Fri, 21 Oct 2016, 20:05 Marton Balint, <cus at passwd.hu> wrote:
>
>>
>> On Thu, 20 Oct 2016, Michael Niedermayer wrote:
>>
>> > On Tue, Oct 18, 2016 at 05:36:27PM -0700, Carlos Fernandez Sanz wrote:
>> >> From: Carlos Fernandez <carlos at ccextractor.org>
>> >>
>> >> Signed-off-by: Carlos Fernandez <carlos at ccextractor.org>
>> >> ---
>> >>  libavcodec/avcodec.h    | 1 +
>> >>  libavcodec/codec_desc.c | 6 ++++++
>> >>  2 files changed, 7 insertions(+)
>> >
>> > Reviewed-by: Michael
>> > LGTM
>>
>> Pushed with a minor whitespace fix.
>>
>> Regards,
>> Marton
>> _______________________________________________
>> ffmpeg-devel mailing list
>> ffmpeg-devel at ffmpeg.org
>> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
>
> So all the objections to this patchset are now irrelevant are they?

No they are not, but in this case the people objecting failed to 
specifically explain what is wrong with the patch and how can it be 
improved to be acceptable.

BTW I only pushed patch 1 & 2, which I find even less questionable.

Regards,
Marton