On 2018-09-06 19:39, Sigríður Regína Sigurþórsdóttir wrote:
> +    if (s->metadata_header_padding) {
> +        if (s->metadata_header_padding == 1)
> +            s->metadata_header_padding++;
> +        put_ebml_void(pb, s->metadata_header_padding);
> +    }

Unfortunately I was forced to make the default -1 so you want to check
that the value is greater than 0 rather than just true.

Furthermore I think you will still want to add to Changelog making a
note that the matroska muxer will now listen to metadata_header_padding.
 That may also want a micro version bump so that library users can check.

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to