On 02/28/2012 06:25 AM, Anton Khirnov wrote:

> ---
>  libavformat/mp3enc.c |  125 
> +++++++++++++++++++++++++++++++++++++++++++++++---
>  1 files changed, 118 insertions(+), 7 deletions(-)
> 
> diff --git a/libavformat/mp3enc.c b/libavformat/mp3enc.c
> index ab20c29..63ce628 100644
> --- a/libavformat/mp3enc.c
> +++ b/libavformat/mp3enc.c
[...]
> +    if (mp3 && mp3->pics_to_write) {
> +            av_log(s, AV_LOG_WARNING, "No packets were sent for some of the "
> +                   "video streams.\n");


This could be confusing to a user. maybe "attached pictures" instead of
"video streams"?

Other than that the patch looks great.

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

Reply via email to