[FFmpeg-cvslog] r19173 - trunk/libavformat/adtsenc.c

Måns Rullgård mans
Fri Jun 12 22:28:31 CEST 2009


Alex Converse <alex.converse at gmail.com> writes:

> 2009/6/12 M?ns Rullg?rd <mans at mansr.com>:
>> alexc <subversion at mplayerhq.hu> writes:
>>
>>> Author: alexc
>>> Date: Fri Jun 12 21:49:46 2009
>>> New Revision: 19173
>>>
>>> Log:
>>> Do not include libavcodec/internal.h in the ADTS muxer.
>>>
>>> Modified:
>>> ? ?trunk/libavformat/adtsenc.c
>>>
>>> Modified: trunk/libavformat/adtsenc.c
>>> ==============================================================================
>>> --- trunk/libavformat/adtsenc.c ? ? ? Fri Jun 12 19:39:56 2009 ? ? ? ?(r19172)
>>> +++ trunk/libavformat/adtsenc.c ? ? ? Fri Jun 12 21:49:46 2009 ? ? ? ?(r19173)
>>> @@ -22,7 +22,6 @@
>>>
>>> ?#include "libavcodec/get_bits.h"
>>> ?#include "libavcodec/put_bits.h"
>>> -#include "libavcodec/internal.h"
>>> ?#include "libavcodec/mpeg4audio.h"
>>> ?#include "avformat.h"
>>>
>>> @@ -64,7 +63,7 @@ static int decode_extradata(AVFormatCont
>>> ? ? ? ? ?return -1;
>>> ? ? ?}
>>> ? ? ?if (get_bits(&gb, 1)) {
>>> - ? ? ? ?ff_log_missing_feature(s, "Signaled SBR or PS", 0);
>>> + ? ? ? ?av_log(s, AV_LOG_ERROR, "Signaled SBR or PS is not supported\n");
>>
>> Why did you do that?
>>
>
> libavformat lacks ff_log_missing_feature

Why can't it call the lavc one?  It already depends on lavc.

This is internal.h crackdown is getting silly.

-- 
M?ns Rullg?rd
mans at mansr.com



More information about the ffmpeg-cvslog mailing list