Option: rename getChild to getChildAtIndex

Clement Champetier · Clement Champetier · commit 7c180d6b09ff · 2014-10-09T14:49:35.000+02:00
diff --git a/app/optionChecker/optionChecker.cpp b/app/optionChecker/optionChecker.cpp
@@ -48,16 +48,16 @@ void displayOptions( const std::vector<avtranscoder::Option>& options )
 			std::cout << "Default choice index: " << option.getDefaultChildIndex() << std::endl;
 			for(size_t i = 0; i < option.getChilds().size(); ++i )
 				std::cout << "Choice " << i << ": " << 
-					option.getChild( i ).getName() << " // " << 
-					option.getChild( i ).getHelp() << std::endl;
+					option.getChildAtIndex( i ).getName() << " // " << 
+					option.getChildAtIndex( i ).getHelp() << std::endl;
 		}
 		else if( option.getType() == avtranscoder::eOptionBaseTypeGroup )
 		{
 			std::cout << "Nb choices: " << option.getChilds().size() << std::endl;
 			for(size_t i = 0; i < option.getChilds().size(); ++i )
 				std::cout << "Element " << i << ": " << 
-					option.getChild( i ).getName() << " // " <<
-					option.getChild( i ).getDefaultValueBool() << std::endl;
+					option.getChildAtIndex( i ).getName() << " // " <<
+					option.getChildAtIndex( i ).getDefaultValueBool() << std::endl;
 		}
 	}
 }
diff --git a/src/AvTranscoder/option/Option.hpp b/src/AvTranscoder/option/Option.hpp
@@ -73,7 +73,7 @@ class Option
 	// array of childs
 	bool hasChild() const { return ! _childOptions.empty(); }
 	const std::vector<Option>& getChilds() { return _childOptions; }
-	const Option& getChild( size_t index ) { return _childOptions.at( index ); }
+	const Option& getChildAtIndex( const size_t index ) const { return _childOptions.at( index ); }
 	int getDefaultChildIndex() const { return _defaultChildIndex; }
 	
 	void setDefaultChildIndex( size_t index ) { _defaultChildIndex = index; }

Original file line number	Diff line number	Diff line change
`@@ -48,16 +48,16 @@ void displayOptions( const std::vector<avtranscoder::Option>& options )`
`48`	`48`	`std::cout << "Default choice index: " << option.getDefaultChildIndex() << std::endl;`
`49`	`49`	`for(size_t i = 0; i < option.getChilds().size(); ++i )`
`50`	`50`	`std::cout << "Choice " << i << ": " <<`
`51`		`- option.getChild( i ).getName() << " // " <<`
`52`		`- option.getChild( i ).getHelp() << std::endl;`
	`51`	`+ option.getChildAtIndex( i ).getName() << " // " <<`
	`52`	`+ option.getChildAtIndex( i ).getHelp() << std::endl;`
`53`	`53`	`}`
`54`	`54`	`else if( option.getType() == avtranscoder::eOptionBaseTypeGroup )`
`55`	`55`	`{`
`56`	`56`	`std::cout << "Nb choices: " << option.getChilds().size() << std::endl;`
`57`	`57`	`for(size_t i = 0; i < option.getChilds().size(); ++i )`
`58`	`58`	`std::cout << "Element " << i << ": " <<`
`59`		`- option.getChild( i ).getName() << " // " <<`
`60`		`- option.getChild( i ).getDefaultValueBool() << std::endl;`
	`59`	`+ option.getChildAtIndex( i ).getName() << " // " <<`
	`60`	`+ option.getChildAtIndex( i ).getDefaultValueBool() << std::endl;`
`61`	`61`	`}`
`62`	`62`	`}`
`63`	`63`	`}`