On 05/12/2011 01:56 AM, Anton Khirnov wrote:

> From: Aurelien Jacobs <[email protected]>
> 
> Signed-off-by: Aurelien Jacobs <[email protected]>
> Signed-off-by: Anton Khirnov <[email protected]>
> ---
>  libavformat/matroskaenc.c |    4 ++++
>  1 files changed, 4 insertions(+), 0 deletions(-)
> 
> diff --git a/libavformat/matroskaenc.c b/libavformat/matroskaenc.c
> index 4aa4caf..1bbabc9 100644
> --- a/libavformat/matroskaenc.c
> +++ b/libavformat/matroskaenc.c
> @@ -614,6 +614,10 @@ static int mkv_write_tracks(AVFormatContext *s)
>  
>              case AVMEDIA_TYPE_SUBTITLE:
>                  put_ebml_uint(pb, MATROSKA_ID_TRACKTYPE, 
> MATROSKA_TRACK_TYPE_SUBTITLE);
> +                if (!native_id) {
> +                    av_log(s, AV_LOG_ERROR, "Subtitle codec %d is not 
> supported.\n", codec->codec_id);
> +                    return AVERROR(ENOSYS);
> +                }
>                  break;
>              default:
>                  av_log(s, AV_LOG_ERROR, "Only audio, video, and subtitles 
> are supported for Matroska.\n");


looks correct.  i assume there is not a non-native subtitle
compatibility mode like there is for video and audio...

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

Reply via email to