[FFmpeg-devel] [PATCH 5/8] lavd: add device capabilities API

Michael Niedermayer michaelni at gmx.at
Sat Apr 5 23:14:41 CEST 2014


On Sat, Apr 05, 2014 at 10:58:37PM +0200, wm4 wrote:
> On Sat, 5 Apr 2014 22:47:37 +0200
> Michael Niedermayer <michaelni at gmx.at> wrote:
> 
> > On Sat, Apr 05, 2014 at 10:17:33PM +0200, Lukasz Marek wrote:
> 
> > > I still think this there should be separate struct to store them in
> > > both cases.
> > > It can be moved to AVFormatContext to simplify API, but avformat
> > > need to know size of the structure. It is avdevice specific so I
> > > don't know if moving it to avformat is good idea. I can add alloc
> > > function in libavdevice, but it will create dependency.
> > > I don't know if it all is worth removing 2 simple API calls.
> > 
> > well, if people are happy with how it is with teh 2 calls, i dont mind
> 
> What people? Is anyone else working on / using the stuff discussed here?

The API discussed here allows to get the capabilities of devices
but they are technically muxers and demuxers, so maybe this API
would at some point in the future also be used for gathering the
capabilities of muxers and demuxers

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Its not that you shouldnt use gotos but rather that you should write
readable code and code with gotos often but not always is less readable
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140405/d7e01634/attachment.asc>


More information about the ffmpeg-devel mailing list