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

[Nicolas George]

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
use.

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.

Regards,

-- 
  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>