ffmpeg_wrapper_file_duration() checks for a string like "Duration: 00:02:12.5, start" in FFmpeg's output. The "start" part isn't always there however, so we should skip that and just look for "Duration: 00:02:12.5".

CommentFileSizeAuthor
#2 ffmpeg_wrapper-duration2.patch1.19 KBzoo33
#1 ffmpeg_wrapper-duration.patch693 byteszoo33

Comments

zoo33’s picture

zoo33 commented
StatusFileSize
new ffmpeg_wrapper-duration.patch693 bytes

The patch...

zoo33’s picture

zoo33 commented
StatusFileSize
new ffmpeg_wrapper-duration2.patch1.19 KB

The search pattern for the duration text had some additional issues, so I rewrote it.
I also changed the code which converts time codes to seconds, as it returned incorrect values.

zoo33’s picture

zoo33 commented
Status: Active » Needs review
zoo33’s picture

zoo33 commented
Status: Needs review » Fixed

Committed.

zoo33’s picture

zoo33 commented
Version: 6.x-1.x-dev » 5.x-1.x-dev
Status: Fixed » Patch (to be ported)
arthurf’s picture

arthurf commented
Status: Patch (to be ported) » Fixed

Patch is ported back to D5

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.