0023264: cppcheck warning: Logical conjunction always evaluates to false
authorPawel <pawel-kowalski@wp.pl>
Thu, 12 Jul 2012 08:17:38 +0000 (10:17 +0200)
committerPawel Kowalski <pawel-kowalski@wp.pl>
Fri, 20 Jul 2012 12:54:16 +0000 (16:54 +0400)
Replaced the '&&' operator with '||'. Otherwise the logical conjunction always evaluates to false.

src/Visual3d/Visual3d_Light.cxx

index c2f4b04..549ceaa 100755 (executable)
@@ -140,11 +140,11 @@ MyType (Visual3d_TOLS_POSITIONAL) {
                Visual3d_LightDefinitionError::Raise
                        ("Bad value for LightAttenuation");
 
-       if ( (Fact1 < 0.0) && (Fact1 > 1.0) )
+       if ( (Fact1 < 0.0) || (Fact1 > 1.0) )
                Visual3d_LightDefinitionError::Raise
                        ("Bad value for LightAttenuation");
 
-       if ( (Fact2 < 0.0) && (Fact2 > 1.0) )
+       if ( (Fact2 < 0.0) || (Fact2 > 1.0) )
                Visual3d_LightDefinitionError::Raise
                        ("Bad value for LightAttenuation");
 
@@ -186,11 +186,11 @@ MyType (Visual3d_TOLS_SPOT) {
                Visual3d_LightDefinitionError::Raise
                        ("Bad value for LightAttenuation");
 
-       if ( (Fact1 < 0.0) && (Fact1 > 1.0) )
+       if ( (Fact1 < 0.0) || (Fact1 > 1.0) )
                Visual3d_LightDefinitionError::Raise
                        ("Bad value for LightAttenuation");
 
-       if ( (Fact2 < 0.0) && (Fact2 > 1.0) )
+       if ( (Fact2 < 0.0) || (Fact2 > 1.0) )
                Visual3d_LightDefinitionError::Raise
                        ("Bad value for LightAttenuation");
 
@@ -378,7 +378,7 @@ void Visual3d_Light::SetAngle (const Standard_Real AngleCone) {
 
 void Visual3d_Light::SetAttenuation1 (const Standard_Real Fact1) {
 
-       if ( (Fact1 < 0.0) && (Fact1 > 1.0) )
+       if ( (Fact1 < 0.0) || (Fact1 > 1.0) )
                Visual3d_LightDefinitionError::Raise
                        ("Bad value for LightAttenuation");
 
@@ -399,7 +399,7 @@ void Visual3d_Light::SetAttenuation1 (const Standard_Real Fact1) {
 
 void Visual3d_Light::SetAttenuation2 (const Standard_Real Fact2) {
 
-       if ( (Fact2 < 0.0) && (Fact2 > 1.0) )
+       if ( (Fact2 < 0.0) || (Fact2 > 1.0) )
                Visual3d_LightDefinitionError::Raise
                        ("Bad value for LightAttenuation");