Re: [FFmpeg-devel] [PATCH] Stream specifier enhancement

2015-10-28 Thread Bodecs Bela
ping 2015.10.18. 14:06 keltezéssel, Bodecs Bela írta: Dear Marton Balint, see may comments below. 2015.10.18. 1:10 keltezéssel, Marton Balint írta: On Thu, 15 Oct 2015, Bodecs Bela wrote: [...] My enhancement does not alter the current behaviour in any way. Are you sure? What if

Re: [FFmpeg-devel] [PATCH] Stream specifier enhancement

2015-10-28 Thread Michael Niedermayer
On Sun, Oct 18, 2015 at 02:06:12PM +0200, Bodecs Bela wrote: > Dear Marton Balint, > > see may comments below. > > 2015.10.18. 1:10 keltezéssel, Marton Balint írta: > > > >On Thu, 15 Oct 2015, Bodecs Bela wrote: > > > >[...] > > > My enhancement does not alter the current behaviour in any

Re: [FFmpeg-devel] [PATCH] Stream specifier enhancement

2015-10-28 Thread Marton Balint
On Sun, 18 Oct 2015, Bodecs Bela wrote: [...] It can be anything if it does not change existing behaviour, a complex specifier can be split to basic specifiers without worrying about the syntax of the basic specifier and if there is a well defined rule for escaping special characters. Also

Re: [FFmpeg-devel] [PATCH] Stream specifier enhancement

2015-10-18 Thread Bodecs Bela
Dear Marton Balint, see may comments below. 2015.10.18. 1:10 keltezéssel, Marton Balint írta: On Thu, 15 Oct 2015, Bodecs Bela wrote: [...] My enhancement does not alter the current behaviour in any way. Are you sure? What if somebody matched a semicolon, or a string which contained a

Re: [FFmpeg-devel] [PATCH] Stream specifier enhancement

2015-10-17 Thread Marton Balint
On Thu, 15 Oct 2015, Bodecs Bela wrote: [...] My enhancement does not alter the current behaviour in any way. Are you sure? What if somebody matched a semicolon, or a string which contained a semicolon in a metadata value? E.g.: m:timecode:00:00:00:00 This question still stands. Will

Re: [FFmpeg-devel] [PATCH] Stream specifier enhancement

2015-10-15 Thread Bodecs Bela
Dear Marton Balint, thank you for your feedback. See my comments below. 2015.10.14. 1:20 keltezéssel, Marton Balint írta: On Tue, 13 Oct 2015, Bodecs Bela wrote: Dear Marton Balint, 2015.10.12. 23:07 keltezéssel, Marton Balint írta: On Mon, 12 Oct 2015, Bodecs Bela wrote: Dear All,

Re: [FFmpeg-devel] [PATCH] Stream specifier enhancement

2015-10-13 Thread Bodecs Bela
Dear Marton Balint, 2015.10.12. 23:07 keltezéssel, Marton Balint írta: On Mon, 12 Oct 2015, Bodecs Bela wrote: Dear All, currently stream specifiers may contain stream index, stream type, stream id, program id, metadata key/value es usable config settings. But you can not combine them. In

Re: [FFmpeg-devel] [PATCH] Stream specifier enhancement

2015-10-13 Thread Marton Balint
On Tue, 13 Oct 2015, Bodecs Bela wrote: Dear Marton Balint, 2015.10.12. 23:07 keltezéssel, Marton Balint írta: On Mon, 12 Oct 2015, Bodecs Bela wrote: Dear All, currently stream specifiers may contain stream index, stream type, stream id, program id, metadata key/value es usable config

[FFmpeg-devel] [PATCH] Stream specifier enhancement

2015-10-12 Thread Bodecs Bela
Dear All, currently stream specifiers may contain stream index, stream type, stream id, program id, metadata key/value es usable config settings. But you can not combine them. In some case, mainly working with multiprogram mpeg-ts containers as input, this feature would be handy. This patch

Re: [FFmpeg-devel] [PATCH] Stream specifier enhancement

2015-10-12 Thread Marton Balint
On Mon, 12 Oct 2015, Bodecs Bela wrote: Dear All, currently stream specifiers may contain stream index, stream type, stream id, program id, metadata key/value es usable config settings. But you can not combine them. In some case, mainly working with multiprogram mpeg-ts containers as input,