[Erp5-report] r45469 priscila.manhaes - /erp5/trunk/utils/cloudooo.handler.ffmpeg/src/cloud...

[nobody at svn.erp5.org]

Author: priscila.manhaes
Date: Thu Apr 14 20:33:07 2011
New Revision: 45469

URL: http://svn.erp5.org?rev=45469&view=rev
Log:
refactor to remove index that do not need anymore

Modified:
    erp5/trunk/utils/cloudooo.handler.ffmpeg/src/cloudooo/handler/ffmpeg/handler.py

Modified: erp5/trunk/utils/cloudooo.handler.ffmpeg/src/cloudooo/handler/ffmpeg/handler.py
URL: http://svn.erp5.org/erp5/trunk/utils/cloudooo.handler.ffmpeg/src/cloudooo/handler/ffmpeg/handler.py?rev=45469&r1=45468&r2=45469&view=diff
==============================================================================
--- erp5/trunk/utils/cloudooo.handler.ffmpeg/src/cloudooo/handler/ffmpeg/handler.py [utf8] (original)
+++ erp5/trunk/utils/cloudooo.handler.ffmpeg/src/cloudooo/handler/ffmpeg/handler.py [utf8] Thu Apr 14 20:33:07 2011
@@ -63,8 +63,8 @@ class Handler(object):
                output_url]
     # XXX ffmpeg has a bug that needs this options to work with webm format
     if destination_format == "webm":
+      command.insert(3, "32k")
       command.insert(3, "-ab")
-      command.insert(4, "32k")
     try:
       stdout, stderr = Popen(command,
                              stdout=PIPE,
@@ -106,11 +106,9 @@ class Handler(object):
                self.input.getUrl(),
                "-y",
                output_url]
-    index = 3
     for metadata in metadata_dict:
-      command.insert(index, "-metadata")
-      command.insert(index+1, "%s=%s"%(metadata, metadata_dict[metadata]))
-      index += 2
+      command.insert(3, "%s=%s"%(metadata, metadata_dict[metadata]))
+      command.insert(3, "-metadata")
     try:
       stdout, stderr = Popen(command,
                              stdout=PIPE,