[vlc-devel] [PATCH 1/2] [RFC] core: add a demux-filter similar to stream-filter for demuxers

[Steve Lhomme]

On Fri, Apr 22, 2016 at 9:47 PM, Francois Cartegnie <fcvlcdev at free.fr> wrote:
> Le 22/04/2016 17:57, Steve Lhomme a écrit :
>> it allows filtering the Demux calls to the underlying muxer
>> it can be chained like stream-filters
>> internally it's a regular demuxer_t with the p_source demuxer set
>>
>> the base value of "demux-filter" is set in libvlc, each input_thread_t has
>> their own copy
>
> I'm not fan of "filter" if it does not.
> If that's just an anchor point for other filters, that could be a
> demux-forward, demux-output, demux-helper, or something else.

In my case I'm just filtering some of the calls. But technically it's
a demuxer wrapper. After you do whatever you want in the wrapper.
I used the filter name because of the possible chaining like with
stream filters. I don't have a strong opinion on the name it should
have.

>>  #define DEMUX_TITLE N_("Demuxers")
>>  #define DEMUX_HELP N_( "Demuxers are used to separate audio and video streams." )
>>
>> @@ -219,6 +224,7 @@ static const struct config_category_t categories_array[] =
>>      { SUBCAT_INPUT_ACODEC, ADEC_TITLE, ADEC_HELP },
>>      { SUBCAT_INPUT_SCODEC, SDEC_TITLE, SDEC_HELP },
>>      { SUBCAT_INPUT_STREAM_FILTER, STREAM_FILTER_TITLE, STREAM_FILTER_HELP },
>> +    { SUBCAT_INPUT_DEMUX_FILTER, DEMUX_FILTER_TITLE, DEMUX_FILTER_HELP },
>>
>
> Internal usage only. Does it need to be visible/listed ?

Mine is internal not meant to be set by the user, but maybe at some
point someone will want a demuxer wrapper.

> Francois
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel