Hi,

On Thu, Jun 2, 2011 at 10:48 PM, Anton Khirnov <[email protected]> wrote:
> +    if (!s->framerate)
> +        switch (s->standard) {
> +        case PAL:   s->framerate = av_strdup("pal");  break;
> +        case NTSC:  s->framerate = av_strdup("ntsc"); break;
> +        case SECAM: s->framerate = av_strdup("25");   break;
> +        default:
> +            av_log(s1, AV_LOG_ERROR, "Unknown standard.\n");
> +            ret = AVERROR(EINVAL);
> +            goto out;
> +        }
>     if ((ret = av_parse_video_rate(&fps, s->framerate)) < 0) {
>         av_log(s1, AV_LOG_ERROR, "Couldn't parse framerate.\n");
>         goto out;

Missing else here? Looks good otherwise.

Ronald
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to