[FFmpeg-trac] #6533(avformat:reopened): Invalid EXT-X-TARGETDURATION in HLS

FFmpeg trac at avcodec.org
Thu Aug 10 13:45:43 EEST 2017


#6533: Invalid EXT-X-TARGETDURATION in HLS
-------------------------------------+------------------------------------
             Reporter:  tonn81       |                    Owner:
                 Type:  defect       |                   Status:  reopened
             Priority:  normal       |                Component:  avformat
              Version:  unspecified  |               Resolution:
             Keywords:  hls          |               Blocked By:
             Blocking:               |  Reproduced by developer:  0
Analyzed by developer:  0            |
-------------------------------------+------------------------------------

Comment (by stevenliu):

 Replying to [comment:17 tonn81]:
 > @stevenliu, do you see the problem I have and the way I see to fix the
 issue?
 >
 > It seems you don't see the whole picture and falling to technical
 details ignoring all the arguments I provided.
 >
 > Is it trolling?
 >
 > The problem is that (because of HLS segmenter generating chunks slightly
 longer than requested) EXT_X_TARGETDURATION value is too high.
 >
 > That makes EXT_X_TARGETDURATION match the specification but not precise
 and not matching users needs.
 >
 > Now if chunk duration is more than `4.001` (or `4.0000001` after your
 change), it would result in `EXT_X_TARGETDURATION = 5`.
 >
 > I propose if chunk duration is < `4.5` to have `EXT_X_TARGETDURATION =
 4` and if chunk duration is >= `4.5` to have `EXT_X_TARGETDURATION = 5`.
 >
 > That would require exactly to change `get_int_from_double(en->duration)`
 to `(int)round(en->duration)`.

 i have read your mind and all, let me make sure what I understand of you.

 You mean  segment duration is x,  if x = 4.400 then TARGETDURATION = 4 ,
 Is that your mean?

--
Ticket URL: <https://trac.ffmpeg.org/ticket/6533#comment:19>
FFmpeg <https://ffmpeg.org>
FFmpeg issue tracker


More information about the FFmpeg-trac mailing list