On Thu, Sep 11, 2014 at 11:53:57PM +0200, Michael Niedermayer wrote:
> On Thu, Sep 11, 2014 at 09:13:29PM +0200, Clément Bœsch wrote:
> > ---
> > libavfilter/vf_subtitles.c | 4 +++-
> > 1 file changed, 3 insertions(+), 1 deletion(-)
> >
> > diff --git a/libavfilter/vf_subtitles.c b/libavfilter/vf_subtitles.c
> > index 4fa1174..44273cb 100644
> > --- a/libavfilter/vf_subtitles.c
> > +++ b/libavfilter/vf_subtitles.c
> > @@ -80,7 +80,9 @@ static const int ass_libavfilter_log_level_map[] = {
> >
> > static void ass_log(int ass_level, const char *fmt, va_list args, void
> > *ctx)
> > {
> > - int level = ass_libavfilter_log_level_map[ass_level];
> > + const int ass_level_clip = av_clip(ass_level, 0,
> > + FF_ARRAY_ELEMS(ass_libavfilter_log_level_map) - 1);
> > + const int level = ass_libavfilter_log_level_map[ass_level_clip];
>
> LGTM
>
> btw, in
> [2] = AV_LOG_WARNING, /* MSGL_WARN */
> [3] = AV_LOG_WARNING, /* <undefined> */
> [4] = AV_LOG_INFO, /* MSGL_INFO */
> [5] = AV_LOG_INFO, /* <undefined> */
> [6] = AV_LOG_VERBOSE, /* MSGL_V */
>
> AV_LOG levels have finer level values than the named identifers
> so it would be possible to use different levels instead of using
> info and warning twice
> Right, but we don't really need that extra complexity with finer grain; the <undefined> entries are basically just unmapped & unused in libass so they won't be called. I'll apply the patchset soon unless someone object. Thanks [...] -- Clément B.
pgpv0EGBXs0qO.pgp
Description: PGP signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
