X-Git-Url: http://www2.svjatoslav.eu/gitweb/?a=blobdiff_plain;f=src%2Fmain%2Fjava%2Feu%2Fsvjatoslav%2Fmeviz%2Fencoder%2Fconverters%2FAvconvVideo.java;h=f0d80bfdd674b1c97bc605c7f4790d164e94143c;hb=bc2cdc4cc3316a5fb66f4372e3708b610c8f68b0;hp=2cda2d8582f8f6f1a14acc3c16e6c7621bdb51e3;hpb=bafe09c26e6ebb5fadf408ecd46bea79cd79e8f1;p=meviz.git

diff --git a/src/main/java/eu/svjatoslav/meviz/encoder/converters/AvconvVideo.java b/src/main/java/eu/svjatoslav/meviz/encoder/converters/AvconvVideo.java
index 2cda2d8..f0d80bf 100644
--- a/src/main/java/eu/svjatoslav/meviz/encoder/converters/AvconvVideo.java
+++ b/src/main/java/eu/svjatoslav/meviz/encoder/converters/AvconvVideo.java
@@ -12,6 +12,7 @@ package eu.svjatoslav.meviz.encoder.converters;
 import java.io.File;
 import java.util.List;
 
+import eu.svjatoslav.meviz.encoder.BitrateParameter;
 import eu.svjatoslav.meviz.encoder.BitrateParameter.bitrate;
 import eu.svjatoslav.meviz.encoder.EncodingOptions;
 
@@ -22,8 +23,12 @@ public class AvconvVideo extends AbstractConverter {
 			final StringBuffer codecParams, final String videoCodec,
 			final String audioCodec) {
 
-		codecParams.append("-acodec " + audioCodec + " -vcodec " + videoCodec
-				+ " ");
+		if (audioCodec == null)
+			codecParams.append("-an ");
+		else
+			codecParams.append("-acodec " + audioCodec + " ");
+
+		codecParams.append("-vcodec " + videoCodec + " ");
 
 		if (videoBitrate != -1)
 			codecParams.append("-b " + videoBitrate + "k ");
@@ -35,36 +40,35 @@ public class AvconvVideo extends AbstractConverter {
 			codecParams.append("-filter:v yadif ");
 	}
 
-	@Override
-	public String getCommand(final File inputFile, final File targetFile,
-			final EncodingOptions options) {
-
-		int videoBitrate = -1;
-		int audioBitrate = -1;
-
-		switch (options.getVideoBitrate()) {
+	private int getAudioBitrateValue(final BitrateParameter.bitrate bitRate) {
+		switch (bitRate) {
 		case LOW:
-			videoBitrate = 1000;
-			audioBitrate = 128;
-			break;
+			return 128;
 
 		case MEDIUM:
-			videoBitrate = 4000;
-			audioBitrate = 192;
-			break;
+			return 160;
 
 		case HIGH:
-			videoBitrate = 40000;
-			audioBitrate = 500;
-			break;
+			return 320;
 
 		case COPY:
-			break;
+			return -1;
+
+		case NONE:
+			return -1;
 
 		default:
-			throw new RuntimeException("Video bitrate: " + options.getVideoBitrate()
+			throw new RuntimeException("Audio bitrate: " + bitRate
 					+ " is not supported.");
 		}
+	}
+
+	@Override
+	public String getCommand(final File inputFile, final File targetFile,
+			final EncodingOptions options) {
+
+		int videoBitrate = getVideoBitrateValue(options.getVideoBitrate());
+		int audioBitrate = getAudioBitrateValue(options.getAudioBitrate());
 
 		// convert
 		final StringBuffer codecParams = new StringBuffer();
@@ -72,10 +76,14 @@ public class AvconvVideo extends AbstractConverter {
 		String videoCodec = "libx264";
 		String audioCodec = "libmp3lame";
 
-		if (options.getVideoBitrate() == bitrate.COPY) {
+		if (options.getVideoBitrate() == bitrate.COPY)
 			videoCodec = "copy";
+
+		if (options.getAudioBitrate() == bitrate.COPY)
 			audioCodec = "copy";
-		}
+
+		if (options.getAudioBitrate() == bitrate.NONE)
+			audioCodec = null;
 
 		if (options.forPortablePlayer) {
 			videoBitrate = 1000;
@@ -102,6 +110,26 @@ public class AvconvVideo extends AbstractConverter {
 		return toList("mkv", "mts", "mp4", "avi", "mpg", "mpeg", "vob", "m4v");
 	}
 
+	private int getVideoBitrateValue(final BitrateParameter.bitrate bitRate) {
+		switch (bitRate) {
+		case LOW:
+			return 1000;
+
+		case MEDIUM:
+			return 4000;
+
+		case HIGH:
+			return 40000;
+
+		case COPY:
+			return -1;
+
+		default:
+			throw new RuntimeException("Video bitrate: " + bitRate
+					+ " is not supported.");
+		}
+	}
+
 	@Override
 	public boolean isTerminalMandatory() {
 		return true;