Planet
navi homePPSaboutscreenshotsdownloaddevelopmentforum

Ignore:
Timestamp:
Oct 3, 2008, 2:44:46 PM (12 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:
1 edited

Legend:

Unmodified
Added
Removed
  • code/trunk/src/core/Executor.cc

    r1786 r1879  
    132132    Executor& Executor::setDescriptionParam(unsigned int param, const std::string& description)
    133133    {
    134         if (param >= 0 && param < MAX_FUNCTOR_ARGUMENTS)
     134        if (param < MAX_FUNCTOR_ARGUMENTS)
    135135        {
    136136            if (!this->bAddedDescriptionParam_[param])
     
    150150    const std::string& Executor::getDescriptionParam(unsigned int param) const
    151151    {
    152         if (param >= 0 && param < MAX_FUNCTOR_ARGUMENTS)
     152        if (param < MAX_FUNCTOR_ARGUMENTS)
    153153            return GetLocalisation(this->descriptionParam_[param]);
    154154
     
    234234    Executor& Executor::setDefaultValue(unsigned int index, const MultiType& param)
    235235    {
    236         if (index >= 0 && index < MAX_FUNCTOR_ARGUMENTS)
     236        if (index < MAX_FUNCTOR_ARGUMENTS)
    237237        {
    238238            this->defaultValue_[index] = param;
Note: See TracChangeset for help on using the changeset viewer.