0023336: The 'strcmp' function returns 0 if corresponding strings are equal.

Checking if the return value from 'strcmp' is '0'.

diff --git a/src/RWStepShape/RWStepShape_RWAngularLocation.cxx b/src/RWStepShape/RWStepShape_RWAngularLocation.cxx
index de17141f6fe153b769f9a153568bdd901c730e14..2a7a3e659704560e5bdbe420b03795810be9f8e7 100755 (executable)
--- a/src/RWStepShape/RWStepShape_RWAngularLocation.cxx
+++ b/src/RWStepShape/RWStepShape_RWAngularLocation.cxx
@@ -71,9 +71,9 @@ void RWStepShape_RWAngularLocation::ReadStep (const Handle(StepData_StepReaderDa
   StepShape_AngleRelator aAngleSelection = StepShape_Small;
   if (data->ParamType (num, 5) == Interface_ParamEnum) {
     Standard_CString text = data->ParamCValue(num, 5);
-    if      (strcmp(text, ".EQUAL.")) aAngleSelection = StepShape_Equal;
-    else if (strcmp(text, ".LARGE.")) aAngleSelection = StepShape_Large;
-    else if (strcmp(text, ".SMALL.")) aAngleSelection = StepShape_Small;
+    if      (strcmp(text, ".EQUAL.") == 0) aAngleSelection = StepShape_Equal;
+    else if (strcmp(text, ".LARGE.") == 0) aAngleSelection = StepShape_Large;
+    else if (strcmp(text, ".SMALL.") == 0) aAngleSelection = StepShape_Small;
     else ach->AddFail("Parameter #5 (angle_selection) has not allowed value");
   }
   else ach->AddFail("Parameter #5 (angle_selection) is not enumeration");

diff --git a/src/RWStepShape/RWStepShape_RWAngularSize.cxx b/src/RWStepShape/RWStepShape_RWAngularSize.cxx
index eaed6f2ba429ba0959dfa9d773fea6e6106fad72..d003d401839f86eef11c63f45628604651fe0860 100755 (executable)
--- a/src/RWStepShape/RWStepShape_RWAngularSize.cxx
+++ b/src/RWStepShape/RWStepShape_RWAngularSize.cxx
@@ -60,9 +60,9 @@ void RWStepShape_RWAngularSize::ReadStep (const Handle(StepData_StepReaderData)&
   StepShape_AngleRelator aAngleSelection = StepShape_Small;
   if (data->ParamType (num, 3) == Interface_ParamEnum) {
     Standard_CString text = data->ParamCValue(num, 3);
-    if      (strcmp(text, ".EQUAL.")) aAngleSelection = StepShape_Equal;
-    else if (strcmp(text, ".LARGE.")) aAngleSelection = StepShape_Large;
-    else if (strcmp(text, ".SMALL.")) aAngleSelection = StepShape_Small;
+    if      (strcmp(text, ".EQUAL.") == 0) aAngleSelection = StepShape_Equal;
+    else if (strcmp(text, ".LARGE.") == 0) aAngleSelection = StepShape_Large;
+    else if (strcmp(text, ".SMALL.") == 0) aAngleSelection = StepShape_Small;
     else ach->AddFail("Parameter #3 (angle_selection) has not allowed value");
   }
   else ach->AddFail("Parameter #3 (angle_selection) is not enumeration");