[FFmpeg-devel] [PATCH] avformat/mov: fix missing line break in messages

Moritz Barsnick barsnick at gmx.net
Fri Jul 10 14:06:43 EEST 2020


On Thu, Jun 25, 2020 at 18:59:58 +0800, mypopy at gmail.com wrote:
> On Thu, Jun 25, 2020 at 6:46 PM Moritz Barsnick <barsnick at gmx.net> wrote:
> >
> > One of them can be triggered by https://samples.ffmpeg.org/F4V/H263_NM_f.mp4.
> >
> > Signed-off-by: Moritz Barsnick <barsnick at gmx.net>
> > ---
> >  libavformat/mov.c | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/libavformat/mov.c b/libavformat/mov.c
> > index adc52de947..40fff5dd7d 100644
> > --- a/libavformat/mov.c
> > +++ b/libavformat/mov.c
> > @@ -894,7 +894,7 @@ static int mov_read_ddts(MOVContext *c, AVIOContext *pb, MOVAtom atom)
> >              (frame_duration_code == 3) ? 4096 : 0;
> >
> >      if (channel_layout_code > 0xff) {
> > -        av_log(c->fc, AV_LOG_WARNING, "Unsupported DTS audio channel layout");
> > +        av_log(c->fc, AV_LOG_WARNING, "Unsupported DTS audio channel layout\n");
> >      }
> >      st->codecpar->channel_layout =
> >              ((channel_layout_code & 0x1) ? AV_CH_FRONT_CENTER : 0) |
> > @@ -5219,7 +5219,7 @@ static int mov_read_elst(MOVContext *c, AVIOContext *pb, MOVAtom atom)
> >          } else {
> >              edit_count = atom.size / elst_entry_size;
> >              if (edit_count * elst_entry_size != atom.size) {
> > -                av_log(c->fc, AV_LOG_WARNING, "ELST atom of %"PRId64" bytes, bigger than %d entries.", atom.size, edit_count);
> > +                av_log(c->fc, AV_LOG_WARNING, "ELST atom of %"PRId64" bytes, bigger than %d entries.\n", atom.size, edit_count);
> >              }
> >          }
> >      }
> LGTM

Ping for push.

Thanks,
Moritz


More information about the ffmpeg-devel mailing list