[FFmpeg-devel] [PATCH 1/2] opt: Add support to query ranges

[Michael Niedermayer]

On Mon, Nov 26, 2012 at 12:32:32AM +0100, Stefano Sabatini wrote:
> On date Sunday 2012-11-25 17:39:36 +0100, Michael Niedermayer encoded:
> > Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
[...]

> > +     */
> > +    struct AVOptionRanges *(*query_ranges)(void *obj, const char *key, int flags);
> 
> What about:
> int *(*query_ranges)(void *obj, struct AVOptionRanges **r, const char *key, int flags);
> 
> so it would be possible to distinguish amongst several failure reasons
> (NOMEM, missing option, others?).

do you think this added complexity is worth it ?

other things fixed

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Those who are too smart to engage in politics are punished by being
governed by those who are dumber. -- Plato 
-------------- 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/20121204/97ec9ab8/attachment.asc>