Transcoder: handle empty filnames only case

Valentin NOEL · Valentin NOEL · commit 4d4ca1ef3292 · 2017-09-21T14:52:52.000+02:00
diff --git a/src/AvTranscoder/transcoder/Transcoder.cpp b/src/AvTranscoder/transcoder/Transcoder.cpp
@@ -335,7 +335,7 @@ ProfileLoader::Profile Transcoder::getProfileFromInputs(const std::vector<InputS
     assert(inputStreamDescArray.size() >= 1);
 
     // Get properties from the first input
-    size_t nonEmptyFileName = 0;
+    size_t nonEmptyFileName = -1;
     for(size_t i = 0; i < inputStreamDescArray.size(); ++i)
     {
         if(!inputStreamDescArray.at(i)._filename.empty())
@@ -344,6 +344,9 @@ ProfileLoader::Profile Transcoder::getProfileFromInputs(const std::vector<InputS
             break;
         }
     }
+    if(nonEmptyFileName == -1)
+        throw std::runtime_error("Cannot handle only empty streams as input");
+
     const InputStreamDesc& inputStreamDesc = inputStreamDescArray.at(nonEmptyFileName);
     InputFile inputFile(inputStreamDesc._filename);
 

Original file line number	Diff line number	Diff line change
`@@ -335,7 +335,7 @@ ProfileLoader::Profile Transcoder::getProfileFromInputs(const std::vector<InputS`
`335`	`335`	`assert(inputStreamDescArray.size() >= 1);`
`336`	`336`
`337`	`337`	`// Get properties from the first input`
`338`		`- size_t nonEmptyFileName = 0;`
	`338`	`+ size_t nonEmptyFileName = -1;`
`339`	`339`	`for(size_t i = 0; i < inputStreamDescArray.size(); ++i)`
`340`	`340`	`{`
`341`	`341`	`if(!inputStreamDescArray.at(i)._filename.empty())`
`@@ -344,6 +344,9 @@ ProfileLoader::Profile Transcoder::getProfileFromInputs(const std::vector<InputS`
`344`	`344`	`break;`
`345`	`345`	`}`
`346`	`346`	`}`
	`347`	`+ if(nonEmptyFileName == -1)`
	`348`	`+ throw std::runtime_error("Cannot handle only empty streams as input");`
	`349`	`+`
`347`	`350`	`const InputStreamDesc& inputStreamDesc = inputStreamDescArray.at(nonEmptyFileName);`
`348`	`351`	`InputFile inputFile(inputStreamDesc._filename);`
`349`	`352`