On Oct 3, 2013, at 11:33 PM, Vittorio Giovara <[email protected]> 
wrote:

> On Wed, Oct 2, 2013 at 10:35 PM, Diego Biurrun <[email protected]> wrote:
>>> +AVCodec ff_libx265_encoder = {
>>> +    .name           = "libx265",
>>> +    .type           = AVMEDIA_TYPE_VIDEO,
>>> +    .id             = AV_CODEC_ID_HEVC,
>>> +    .init           = libx265_encode_init,
>>> +    .encode2        = libx265_encode_frame,
>>> +    .close          = libx265_encode_close,
>>> +    .pix_fmts       = (const enum AVPixelFormat[]){ AV_PIX_FMT_YUV420P, 
>>> AV_PIX_FMT_NONE },
>>> +    .priv_data_size = sizeof(libx265Context),
>>> +    .long_name      = NULL_IF_CONFIG_SMALL("libx265 H.265 / HEVC"),
>> 
>> Move .long_name after .name.
> 
> Also please use HEVC name first if you don't mind.
> Vittorio

Just FWIW, the libx264 wrapper uses:

NULL_IF_CONFIG_SMALL("libx264 H.264 / AVC / MPEG-4 AVC / MPEG-4 part 10")

and the H.264 decoder uses:

NULL_IF_CONFIG_SMALL("H.264 / AVC / MPEG-4 AVC / MPEG-4 part 10")

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

Reply via email to