ProfileLoader: getters to profiles are const functions

Clement Champetier · Clement Champetier · commit 4e84007ddb99 · 2015-03-18T14:27:01.000+01:00
diff --git a/src/AvTranscoder/profile/ProfileLoader.cpp b/src/AvTranscoder/profile/ProfileLoader.cpp
@@ -102,16 +102,16 @@ void ProfileLoader::loadProfile( const Profile& profile )
 		throw std::runtime_error( "Warning: The profile " + profile.find( constants::avProfileIdentificator )->second + " is invalid. It will not be loaded." );
 }
 
-const ProfileLoader::Profiles& ProfileLoader::getProfiles()
+const ProfileLoader::Profiles& ProfileLoader::getProfiles() const
 {
 	return _profiles;
 }
 
-ProfileLoader::Profiles ProfileLoader::getFormatProfiles()
+ProfileLoader::Profiles ProfileLoader::getFormatProfiles() const
 {
 	Profiles profiles;
 
-	for( Profiles::iterator it = _profiles.begin(); it != _profiles.end(); ++it )
+	for( Profiles::const_iterator it = _profiles.begin(); it != _profiles.end(); ++it )
 	{
 		if( (*it).find( constants::avProfileType )->second == constants::avProfileTypeFormat )
 			profiles.push_back( *it );
@@ -120,11 +120,11 @@ ProfileLoader::Profiles ProfileLoader::getFormatProfiles()
 	return profiles;
 }
 
-ProfileLoader::Profiles ProfileLoader::getVideoProfiles()
+ProfileLoader::Profiles ProfileLoader::getVideoProfiles() const
 {
 	Profiles profiles;
 
-	for( Profiles::iterator it = _profiles.begin(); it != _profiles.end(); ++it )
+	for( Profiles::const_iterator it = _profiles.begin(); it != _profiles.end(); ++it )
 	{
 		if( (*it).find( constants::avProfileType )->second == constants::avProfileTypeVideo )
 			profiles.push_back( *it );
@@ -133,11 +133,11 @@ ProfileLoader::Profiles ProfileLoader::getVideoProfiles()
 	return profiles;
 }
 
-ProfileLoader::Profiles ProfileLoader::getAudioProfiles()
+ProfileLoader::Profiles ProfileLoader::getAudioProfiles() const
 {
 	Profiles profiles;
 
-	for( Profiles::iterator it = _profiles.begin(); it != _profiles.end(); ++it )
+	for( Profiles::const_iterator it = _profiles.begin(); it != _profiles.end(); ++it )
 	{
 		if( (*it).find( constants::avProfileType )->second == constants::avProfileTypeAudio )
 			profiles.push_back( *it );
@@ -159,7 +159,7 @@ ProfileLoader::Profile& ProfileLoader::getProfile( const std::string& avProfileI
 }
 
 
-bool ProfileLoader::checkFormatProfile( const Profile& profileToCheck )
+bool ProfileLoader::checkFormatProfile( const Profile& profileToCheck ) const
 {
 	bool isValid = true;
 
@@ -174,7 +174,7 @@ bool ProfileLoader::checkFormatProfile( const Profile& profileToCheck )
 	return isValid;
 }
 
-bool ProfileLoader::checkVideoProfile( const Profile& profileToCheck )
+bool ProfileLoader::checkVideoProfile( const Profile& profileToCheck ) const
 {
 	bool isValid = true;
 
@@ -189,7 +189,7 @@ bool ProfileLoader::checkVideoProfile( const Profile& profileToCheck )
 	return isValid;
 }
 
-bool ProfileLoader::checkAudioProfile( const Profile& profileToCheck )
+bool ProfileLoader::checkAudioProfile( const Profile& profileToCheck ) const
 {
 	bool isValid = true;
 
diff --git a/src/AvTranscoder/profile/ProfileLoader.hpp b/src/AvTranscoder/profile/ProfileLoader.hpp
@@ -57,18 +57,18 @@ class AvExport ProfileLoader
 	 */
 	void loadProfile( const Profile& profile );
 
-	const Profiles& getProfiles();
+	const Profiles& getProfiles() const;
 
-	Profiles getFormatProfiles();
-	Profiles getVideoProfiles();
-	Profiles getAudioProfiles();
+	Profiles getFormatProfiles() const;
+	Profiles getVideoProfiles() const;
+	Profiles getAudioProfiles() const;
 
 	Profile& getProfile( const std::string& avProfileIdentificator );
 
 private:
-	bool checkFormatProfile( const Profile& profileToCheck );
-	bool checkVideoProfile( const Profile& profileToCheck );
-	bool checkAudioProfile( const Profile& profileToCheck );
+	bool checkFormatProfile( const Profile& profileToCheck ) const;
+	bool checkVideoProfile( const Profile& profileToCheck ) const;
+	bool checkAudioProfile( const Profile& profileToCheck ) const;
 
 private:
 	Profiles _profiles;

Original file line number	Diff line number	Diff line change
`@@ -102,16 +102,16 @@ void ProfileLoader::loadProfile( const Profile& profile )`
`102`	`102`	`throw std::runtime_error( "Warning: The profile " + profile.find( constants::avProfileIdentificator )->second + " is invalid. It will not be loaded." );`
`103`	`103`	`}`
`104`	`104`
`105`		`-const ProfileLoader::Profiles& ProfileLoader::getProfiles()`
	`105`	`+const ProfileLoader::Profiles& ProfileLoader::getProfiles() const`
`106`	`106`	`{`
`107`	`107`	`return _profiles;`
`108`	`108`	`}`
`109`	`109`
`110`		`-ProfileLoader::Profiles ProfileLoader::getFormatProfiles()`
	`110`	`+ProfileLoader::Profiles ProfileLoader::getFormatProfiles() const`
`111`	`111`	`{`
`112`	`112`	`Profiles profiles;`
`113`	`113`
`114`		`- for( Profiles::iterator it = _profiles.begin(); it != _profiles.end(); ++it )`
	`114`	`+ for( Profiles::const_iterator it = _profiles.begin(); it != _profiles.end(); ++it )`
`115`	`115`	`{`
`116`	`116`	`if( (*it).find( constants::avProfileType )->second == constants::avProfileTypeFormat )`
`117`	`117`	`profiles.push_back( *it );`
`@@ -120,11 +120,11 @@ ProfileLoader::Profiles ProfileLoader::getFormatProfiles()`
`120`	`120`	`return profiles;`
`121`	`121`	`}`
`122`	`122`
`123`		`-ProfileLoader::Profiles ProfileLoader::getVideoProfiles()`
	`123`	`+ProfileLoader::Profiles ProfileLoader::getVideoProfiles() const`
`124`	`124`	`{`
`125`	`125`	`Profiles profiles;`
`126`	`126`
`127`		`- for( Profiles::iterator it = _profiles.begin(); it != _profiles.end(); ++it )`
	`127`	`+ for( Profiles::const_iterator it = _profiles.begin(); it != _profiles.end(); ++it )`
`128`	`128`	`{`
`129`	`129`	`if( (*it).find( constants::avProfileType )->second == constants::avProfileTypeVideo )`
`130`	`130`	`profiles.push_back( *it );`
`@@ -133,11 +133,11 @@ ProfileLoader::Profiles ProfileLoader::getVideoProfiles()`
`133`	`133`	`return profiles;`
`134`	`134`	`}`
`135`	`135`
`136`		`-ProfileLoader::Profiles ProfileLoader::getAudioProfiles()`
	`136`	`+ProfileLoader::Profiles ProfileLoader::getAudioProfiles() const`
`137`	`137`	`{`
`138`	`138`	`Profiles profiles;`
`139`	`139`
`140`		`- for( Profiles::iterator it = _profiles.begin(); it != _profiles.end(); ++it )`
	`140`	`+ for( Profiles::const_iterator it = _profiles.begin(); it != _profiles.end(); ++it )`
`141`	`141`	`{`
`142`	`142`	`if( (*it).find( constants::avProfileType )->second == constants::avProfileTypeAudio )`
`143`	`143`	`profiles.push_back( *it );`
`@@ -159,7 +159,7 @@ ProfileLoader::Profile& ProfileLoader::getProfile( const std::string& avProfileI`
`159`	`159`	`}`
`160`	`160`
`161`	`161`
`162`		`-bool ProfileLoader::checkFormatProfile( const Profile& profileToCheck )`
	`162`	`+bool ProfileLoader::checkFormatProfile( const Profile& profileToCheck ) const`
`163`	`163`	`{`
`164`	`164`	`bool isValid = true;`
`165`	`165`
`@@ -174,7 +174,7 @@ bool ProfileLoader::checkFormatProfile( const Profile& profileToCheck )`
`174`	`174`	`return isValid;`
`175`	`175`	`}`
`176`	`176`
`177`		`-bool ProfileLoader::checkVideoProfile( const Profile& profileToCheck )`
	`177`	`+bool ProfileLoader::checkVideoProfile( const Profile& profileToCheck ) const`
`178`	`178`	`{`
`179`	`179`	`bool isValid = true;`
`180`	`180`
`@@ -189,7 +189,7 @@ bool ProfileLoader::checkVideoProfile( const Profile& profileToCheck )`
`189`	`189`	`return isValid;`
`190`	`190`	`}`
`191`	`191`
`192`		`-bool ProfileLoader::checkAudioProfile( const Profile& profileToCheck )`
	`192`	`+bool ProfileLoader::checkAudioProfile( const Profile& profileToCheck ) const`
`193`	`193`	`{`
`194`	`194`	`bool isValid = true;`
`195`	`195`