On 10/04/14 10:35, Vittorio Giovara wrote:
> +#endif /* CONFIG_OGG_MUXER */
> +
> +#if CONFIG_OGA_MUXER
> +static const AVClass oga_muxer_class = {
> +    .class_name = "Ogg audio muxer",
> +    .item_name  = av_default_item_name,
> +    .option     = options,
> +    .version    = LIBAVUTIL_VERSION_INT,
> +};

To be checked what ogg should do with the audio offsets now. (see the
definition of NEGATIVE_TS)

> +AVOutputFormat ff_oga_muxer = {
> +    .name              = "oga",
> +    .long_name         = NULL_IF_CONFIG_SMALL("Ogg audio"),
> +    .mime_type         = "audio/ogg",
> +    .extensions        = "oga",
> +    .priv_data_size    = sizeof(OGGContext),
> +    .audio_codec       = CONFIG_LIBVORBIS_ENCODER ? AV_CODEC_ID_VORBIS
> +                                                  : AV_CODEC_ID_FLAC,

I'd move opus and spx there as well.

> +    .video_codec       = AV_CODEC_ID_NONE,
> +    .write_header      = ogg_write_header,
> +    .write_packet      = ogg_write_packet,
> +    .write_trailer     = ogg_write_trailer,
> +    .flags             = AVFMT_TS_NEGATIVE,
> +    .priv_class        = &oga_muxer_class,

Beside that shouldn't hurt to add it.

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

Reply via email to