Use of libmp3lame as audio format for ffmpeg

tagliavinid - April 23, 2009 - 07:43
Project:FlashVideo
Version:6.x-1.5-rc2
Component:Code
Category:bug report
Priority:minor
Assigned:attheshow
Status:closed
Description

Line 1046 of source code doesn't work for me

if (strpos($format, 'EA   libmp3lame') !== FALSE) {

I try to execute ffmpeg.exe -formats and the output for the above line is this:

EA    libmp3lame

There are 4 space char between EA and libmp3lame, instead of 3.

For this reason libmp3lame for me is never used.

I used this :

if (strpos($format, 'libmp3lame') !== FALSE) {

But maybe this it's safer
if (preg_match("/EA\s+libmp3lame/mi", $format)

#1

tagliavinid - April 23, 2009 - 08:25

Sorry, this code is better, I forgot a parenthesis in the previous one

if (preg_match("/EA\s+libmp3lame/mi", $format))  {

#2

attheshow - April 24, 2009 - 02:04
Assigned to:Anonymous» attheshow
Status:active» fixed

Committed to dev version. Thanks tagliavinid!

#3

System Message - May 8, 2009 - 02:10
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.