[MPlayer-dev-eng] [PATCH] Allow loading config per stream type.

Benjamin Zores ben at geexbox.org
Mon Jan 7 20:24:50 CET 2008


Jindrich Makovicka a écrit :
> On Sat, 5 Jan 2008 21:45:44 +0200
> Onur Küçük <onur at delipenguen.net> wrote:
> 
>>> [extension.mpeg]
>>> profile-desc="Profile for MPEG files"
>>> vo=x11
>>  I like the idea :)
>>
>>  If it is not done intentionally, how about relaxing profile name
>> checking ? This way it is working for "a.mpeg" but not "A.MPEG" .
>>
>>  Maybe tolower the extension in load_per_extension_config or changing
>> strcmp to strcasecmp in m_config_get_profile, which is IMO better
>> (for mencoder too). Though don't know what others will say for
>> m_config_get_profile change.
> 
> A/V format-specific configuration, like [video.DIV3], [video.avc1] would
> be nice too. It would allow to insert filters like -fspp only for
> formats where it makes some sense and the CPU can handle the load, say,
> enable it for mpeg2, but not for h264 or wmv9.
> 
> Before the filter chain is initialized, the fourccs are already
> available in sh_video->format and sh_audio->format, so it should be
> easy to add such option if the configuration parser is written.

Seems not, too mcuh things are initialized when codecs are known.
Adding configuration profile just after sh_video->codecs and 
sh_audio->codecs are known is not enough.
No idea if sh_video->format is known earlier but i don't really like 
having to manage a table between format id and codec name.

Ben



More information about the MPlayer-dev-eng mailing list