On 27/02/14 21:00, Diego Biurrun wrote:
> On Thu, Feb 27, 2014 at 07:06:08PM +0100, Luca Barbato wrote:
>> They require the full decoder.
>> ---
>>  configure           |  4 ++--
>>  libavcodec/Makefile | 11 ++---------
>>  2 files changed, 4 insertions(+), 11 deletions(-)
> 
> That the parser requires the full decoder seems like the underlying issue
> to me.

I'm chopping h264.c in pieces so hopefully the parser might not depend
in the whole deal anymore.

Factoring out so I don't have to touch the makefile in 3 places.

> 
>> --- a/configure
>> +++ b/configure
>> @@ -1764,7 +1764,7 @@ sipr_decoder_select="lsp"
>>  svq1_encoder_select="aandcttables dsputil hpeldsp mpegvideoenc"
>> -svq3_decoder_select="golomb h264chroma h264dsp h264pred h264qpel hpeldsp 
>> mpegvideo videodsp"
>> +svq3_decoder_select="golomb h264_decoder h264chroma h264dsp h264pred 
>> h264qpel hpeldsp mpegvideo videodsp"
>>  svq3_decoder_suggest="error_resilience zlib"
>> @@ -1851,7 +1851,7 @@ wmv3_vaapi_hwaccel_select="vc1_vaapi_hwaccel"
>>  # parsers
>> -h264_parser_select="golomb h264chroma h264dsp h264pred h264qpel videodsp"
>> +h264_parser_select="golomb h264_decoder h264chroma h264dsp h264pred 
>> h264qpel videodsp"
>>  mpeg4video_parser_select="error_resilience h263dsp mpegvideo"
> 
> If you make these depend on the full H.264 decoder you do not need to
> duplicate the H.264 decoder dependencies here.

Right, I should be able to make it simpler. Changing locally, tell me if
is ok to commit or you want to see them again.


lu


_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to