Incorrect handling of extra parameters for ffmpeg (for vpre fix)

Using the 29 September 2009 build of bd_video, input entered into the 'extra parameters (advanced)' field in the BDV admin page are incorrectly handled in the command sent to ffmpeg.

I needed to fix the (vpre) preset error caused by the latest build of ffmpeg, so I simply added '-vpre libx264-default' to this field. However, this resulted in a command in the form of [ffmpeg command] [normal parameters] [output filename] [extra parameters] [2>&1].

Note that the output filename precedes the extra parameters, this is a big no no for ffmpeg's argument syntax.

I ended up fixing the problem by modifying /modules/bd_video/includes/ffmpeg.inc
Changing line 16 from "$command = array($ffmpeg, $params, $extra_params, '2>&1');"
to "$command = array($ffmpeg, $extra_params, $params, '2>&1');"

Resulting in [ffmpeg command] [extra parameters] [normal parameters] [output filename] [2>&1].

It's kind of a dirty fix, but it worked.