Changes between Initial Version and Version 1 of Ticket #4930, comment 12


Ignore:
Timestamp:
Jul 26, 2016, 5:20:03 PM (3 years ago)
Author:
anssi
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #4930, comment 12

    initial v1  
    55The following patch fixes this issue, and I expect to push that within a day unless any problems surface in my testing: https://github.com/anssih/FFmpeg/commit/2fd2c0bf7478ec481792eab7f42085af0f9e109d
    66
    7 There is also a related issue in avformat_find_stream_info() affecting FFmpeg 3.1 and later which causes some needlessly-long probing of the sample in comment 5, which should be fixed by this patch (which I have submitted for review on ffmpeg-devel@): https://github.com/anssih/FFmpeg/commit/be52a0d8c1ae244411091d57a72fc3e773872f6b
     7There is also a related issue in avformat_find_stream_info() affecting FFmpeg 3.1 and later which causes some needlessly-long probing of the sample in comment 5, which should be fixed by this patch (which I have submitted for review on ffmpeg-devel@): ~~https://github.com/anssih/FFmpeg/commit/be52a0d8c1ae244411091d57a72fc3e773872f6b~~ (edit: fixed version https://github.com/anssih/FFmpeg/commit/16f01f43a6f361580aaa3898b3adb22dd41e2535 )
    88
    99Related: My first solution involved adding support for dynamic streams into the HLS demuxer (propagate new streams from subdemuxer), but as that ultimately turned out to not be necessary I won't be applying that at this time (as it adds complexity and risks regressions without having any sample it helps with). In case that feature or parts of it turn out to be useful in the future I've put the code here: https://github.com/anssih/FFmpeg/commit/68eac58088b27593b28eba33632167ed2ebb23f5