[vlc-devel] [vlc-commits] demux: mkv: support OPUS/EXPERIMENTAL codec ID

[Steve Lhomme]

Le 15/02/2018 à 22:34, Tristan Matthews a écrit :
> On Thu, Feb 15, 2018 at 11:25 AM, Rémi Denis-Courmont <remi at remlab.net> wrote:
>> Le jeudi 15 février 2018, 16:44:29 EET Tristan Matthews a écrit :
>>>> IMO it's not a good move to support temporary versions of a binary format.
>>>> How many versions were there ?
>>> 5 it looks like.
>> (...)
>>> It looks like Moritz did that to incrementally add a few features in
>>> between his initial patch and the final one, namely:
>> (...)
>>
>> Why should we support draft MKV bindings?
> According to the ticket, we already did up until at least VLC 2.2.8,
> this is a regression in 3.0 [1]
>
>> For the sake of the argument, we are
>> not going to support all draft versions of AV1, are we?
> We never did, so no, there would be no point in adding it now.
>
>>>> Can it be used for
>>>> other experiments in the future ?
>>> That seems unlikely.
>> I think we have better things to do that add hacks to support 5 different
>> draft versions of a spec.
> The only hack is not rejecting the CodecID, otherwise of course it
> doesn't make sense to add other special case handling.
>
> I could add a warning?

msg_Info might be a good idea, yes.

>
> Best,
> Tristan
>
> [1] https://trac.videolan.org/vlc/ticket/19660
>
>> --
>> Rémi Denis-Courmont
>> _______________________________________________
>> vlc-devel mailing list
>> To unsubscribe or modify your subscription options:
>> https://mailman.videolan.org/listinfo/vlc-devel
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel