[FFmpeg-trac] #2081(avformat:new): Some MOV chapters parsing broken since 0d96ec1

[FFmpeg]

#2081: Some MOV chapters parsing broken since 0d96ec1
-------------------------------------+-------------------------------------
               Reporter:  Cigaes     |                  Owner:
                   Type:  defect     |                 Status:  new
               Priority:  important  |              Component:  avformat
                Version:             |               Keywords:  mov
  unspecified                        |  chapters regression
             Blocked By:             |               Blocking:
Reproduced by developer:  0          |  Analyzed by developer:  1
-------------------------------------+-------------------------------------
 With the attached sample:
 {{{
 ./ffmpeg_g -i /tmp/chapters.mov
 }}}
 Expected (truncated) outout:
 {{{
 Input #0, mov,mp4,m4a,3gp,3g2,mj2, from '/tmp/chapters.mov':
   Metadata:
     major_brand     : qt
     minor_version   : 537199360
     compatible_brands: qt
     creation_time   : 2012-12-24 00:08:45
   Duration: 00:21:21.96, start: 0.000000, bitrate: 3 kb/s
     Chapter #0.0: start 0.000000, end 0.480000
     Metadata:
       title           :
     Stream #0:0(eng): Video: h264 (avc1 / 0x31637661), 640x480, 1694 kb/s,
 25 fps, 25 tbr, 2500 tbn, 5k tbc
 }}}

 Actual output:
 {{{
 Input #0, mov,mp4,m4a,3gp,3g2,mj2, from '/tmp/chapters.mov':
   Metadata:
     major_brand     : qt
     minor_version   : 537199360
     compatible_brands: qt
     creation_time   : 2012-12-24 00:08:45
   Duration: 00:21:21.96, start: 0.000000, bitrate: 3 kb/s
     Stream #0:0(eng): Video: h264 (avc1 / 0x31637661), 640x480, 1694 kb/s,
 25 fps, 25 tbr, 2500 tbn, 5k tbc
 }}}

 Notice the chapter just after "duration". The change comes from the
 following commit:
 {{{0d96ec1 mov: parse tref atom.}}}
 Reverting it (or just replacing {{{mov_read_tref}}} with
 {{{mov_read_default}}} in
 {{{libavformat/mov.c}}}:{{{mov_default_parse_table}}} restores the correct
 behaviour (but certainly breaks the feature it was adding).

-- 
Ticket URL: <https://ffmpeg.org/trac/ffmpeg/ticket/2081>
FFmpeg <http://ffmpeg.org>
FFmpeg issue tracker