[FFmpeg-devel] [PATCH 2/4] lavd: add device capabilities API

Nicolas George george at nsup.org
Mon Feb 17 17:48:30 CET 2014

Le nonidi 29 pluviôse, an CCXXII, Lukasz M a écrit :
> I did check it quickly and it seems image size is represented as string
> ("640x480" for example)
> I haven't found how range of image sizes should be represented.
> If there is no format defined then I can propose something like that
> "100x100-640x480"
> or
> "100-640x100-480"

IMHO, strings are just not suitable for automated processing: the IMAGE_SIZE
option type is well and good for user interaction, but not for programmatic

If no better idea arrives, I believe, the best course of action is to
implement listing of supported resolutions in the example:

    for (w ... supported frame widths) {
        set width to w;
        for (h ... supported frame heights for w)
            printf("Supported frame size: %d x %d\n", w, h);

and then add a pointer to that in the doxy comments.


  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140217/6635db5e/attachment.asc>

More information about the ffmpeg-devel mailing list