[FFmpeg-devel] [PATCH] ffmpeg: add -devices param

[Michael Niedermayer]

On Tue, May 27, 2014 at 11:17:21PM +0200, Lukasz Marek wrote:
> On 27.05.2014 20:17, Lukasz Marek wrote:
> >On 27 May 2014 20:07, Michael Niedermayer <michaelni at gmx.at
> ><mailto:michaelni at gmx.at>> wrote:
> >
> >    On Tue, May 27, 2014 at 07:52:24PM +0200, Clément Bœsch wrote:
> >     > On Tue, May 27, 2014 at 04:47:20PM +0200, Michael Niedermayer wrote:
> >     > > On Fri, May 23, 2014 at 07:59:53PM +0200, Lukasz Marek wrote:
> >     > > > Ffmpeg mix devices and file formats when -formats option is used.
> >     > > > This commit removes devices from formats list.
> >     > > > Devices can be listed with -devices param.
> >     > > > ---
> >     > > >  cmdutils.c                   |   35
> >    ++++++++++++++++++++++++++++++++---
> >     > > >  cmdutils.h                   |    7 +++++++
> >     > > >  cmdutils_common_opts.h       |    1 +
> >     > > >  doc/fftools-common-opts.texi |   10 +++++-----
> >     > > >  doc/indevs.texi              |    4 ++--
> >     > > >  doc/outdevs.texi             |    4 ++--
> >     > > >  6 files changed, 49 insertions(+), 12 deletions(-)
> >     > >
> >     > > adding "-devices" sounds like a good idea to me
> >     > > not sure either way about ommiting them from formats
> >     > > anyone has any oppinon about that ?
> >     > >
> >     >
> >     > If you change the behaviour of -formats you should maybe bump
> >    micro and
> >     > mention it in the Changelog (it might break some scripts).
> >
> >    maybe its better to leave formats as is then for now, we can always
> >    change it later if theres a reason for it.
> >
> >
> >I had some doubts about it too. I will restore behavior of -formats
> >later and resubmit.
> 
> Updated patch attached. -formats behave as before. -devices list
> devices only.

>  cmdutils.c             |   35 ++++++++++++++++++++++++++++++++---
>  cmdutils.h             |    9 ++++++++-
>  cmdutils_common_opts.h |    1 +
>  doc/indevs.texi        |    4 ++--
>  doc/outdevs.texi       |    4 ++--
>  5 files changed, 45 insertions(+), 8 deletions(-)
> 1cecb6b0083ea20a7b40330cdca86eccb6d1dcaa  0001-ffmpeg-add-devices-param.patch
> From 99fdb5da96061639c61a0a6e2439618ae1a20e3b Mon Sep 17 00:00:00 2001
> From: Lukasz Marek <lukasz.m.luki2 at gmail.com>
> Date: Tue, 27 May 2014 23:04:52 +0200
> Subject: [PATCH] ffmpeg: add -devices param

applied

thx

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

It is what and why we do it that matters, not just one of them.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140528/ce12138c/attachment.asc>