I have improved the regexp used to extract the width and height info from ffmpeg, it's quite a bit more reliable now. Tested a few times and seems to work as expected.

CommentFileSizeAuthor
#1 video_ffmpeg.patch750 bytesmnzaki
video_ffmpeg.patch604 bytesmnzaki

Comments

mnzaki’s picture

mnzaki commented
StatusFileSize
new video_ffmpeg.patch750 bytes

Please ignore the above patch, and see the attached.

trent--’s picture

trent-- commented
Status: Needs review » Reviewed & tested by the community

I can confirm that this fixes errors I had with the module being unable to get the video dimensions.
I am using FFmpeg version 0.6-4:0.6-2ubuntu6.1
Without the patch, this module gets wrong information from a line similar to this in ffmpeg output :
[avi @ 0x9e9420]max_analyze_duration reached

hypertext200’s picture

hypertext200
he/him
Primary language Singhalese
Location New York City
commented
Version: 7.x-1.x-dev » 7.x-2.x-dev
Status: Reviewed & tested by the community » Closed (fixed)