Option: removed unnecessary insert of options without unit to looking for childs

Clement Champetier · Clement Champetier · commit 80ef9d32ee6a · 2016-08-29T12:30:46.000+02:00
diff --git a/src/AvTranscoder/Option.cpp b/src/AvTranscoder/Option.cpp
@@ -255,7 +255,10 @@ void loadOptions(OptionMap& outOptions, void* av_class, int req_flags)
         else
         {
             outOptions.insert(std::make_pair(option.getName(), option));
-            optionUnitToParentName.insert(std::make_pair(option.getUnit(), option.getName()));
+            if(! option.getUnit().empty())
+            {
+                optionUnitToParentName.insert(std::make_pair(option.getUnit(), option.getName()));
+            }
         }
     }
 

Original file line number	Diff line number	Diff line change
`@@ -255,7 +255,10 @@ void loadOptions(OptionMap& outOptions, void* av_class, int req_flags)`
`255`	`255`	`else`
`256`	`256`	`{`
`257`	`257`	`outOptions.insert(std::make_pair(option.getName(), option));`
`258`		`- optionUnitToParentName.insert(std::make_pair(option.getUnit(), option.getName()));`
	`258`	`+ if(! option.getUnit().empty())`
	`259`	`+ {`
	`260`	`+ optionUnitToParentName.insert(std::make_pair(option.getUnit(), option.getName()));`
	`261`	`+ }`
`259`	`262`	`}`
`260`	`263`	`}`
`261`	`264`