Changeset 1879 for code/trunk/src/core/Executor.cc

Timestamp:

Oct 3, 2008, 2:44:46 PM (16 years ago)

Author:

rgrieder

Message:

Removed some always-true statements that gcc issued with the -W option.
There is still a lot of warnings from the Ogre UTFString header about the copy constructor initialisation.
—> We would need to use -W -Wno-unused-parameter -Wno-?. I couldn't find that option directly unfortunately.

File:

• code/trunk/src/core/Executor.cc (modified) (3 diffs)

code/trunk/src/core/Executor.cc

@@ -132 +132 @@
     Executor& Executor::setDescriptionParam(unsigned int param, const std::string& description)
     {
-        if (param >= 0 && param < MAX_FUNCTOR_ARGUMENTS)
+        if (param < MAX_FUNCTOR_ARGUMENTS)
         {
             if (!this->bAddedDescriptionParam_[param])
@@ -150 +150 @@
     const std::string& Executor::getDescriptionParam(unsigned int param) const
     {
-        if (param >= 0 && param < MAX_FUNCTOR_ARGUMENTS)
+        if (param < MAX_FUNCTOR_ARGUMENTS)
             return GetLocalisation(this->descriptionParam_[param]);
 
@@ -234 +234 @@
     Executor& Executor::setDefaultValue(unsigned int index, const MultiType& param)
     {
-        if (index >= 0 && index < MAX_FUNCTOR_ARGUMENTS)
+        if (index < MAX_FUNCTOR_ARGUMENTS)
         {
             this->defaultValue_[index] = param;