OpenCascade/occt
1
0
Fork 0
mirror of https://git.dev.opencascade.org/repos/occt.git synced 2025-04-16 10:08:36 +03:00
Code

0023356: Suspicious assignment inside the condition expression of 'if' operator.

Browse Source 
Changed the assignments in the if-clauses to comparisons.
This commit is contained in:
Pawel 2012-07-26 13:06:55 +02:00 committed by Pawel Kowalski
parent d4a137d527
commit 6d1a5d3a0c
2 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/QABugs/QABugs_11.cxx
View File

@ -724,7 +724,7 @@ static Standard_Integer OCC297 (Draw_Interpretor& di,Standard_Integer /*argc*/,
static Standard_Integer OCC305 (Draw_Interpretor& di,Standard_Integer argc, const char ** argv ) static Standard_Integer OCC305 (Draw_Interpretor& di,Standard_Integer argc, const char ** argv )
{ {
if (argc =! 2) if (argc != 2)
{ {
di <<"Usage : " << argv[0] << " file"<<"\n"; di <<"Usage : " << argv[0] << " file"<<"\n";
return 1; return 1;
@ -1895,7 +1895,7 @@ static Standard_Integer OCC909 (Draw_Interpretor& di, Standard_Integer argc, con
//======================================================================= //=======================================================================
static Standard_Integer OCC921 (Draw_Interpretor& di, Standard_Integer argc, const char ** argv) static Standard_Integer OCC921 (Draw_Interpretor& di, Standard_Integer argc, const char ** argv)
{ {
if (argc =! 2) if (argc != 2)
{ {
di <<"Usage : " << argv[0] << " face"<<"\n"; di <<"Usage : " << argv[0] << " face"<<"\n";
return 1; return 1;
@ -1918,7 +1918,7 @@ static Standard_Integer OCC921 (Draw_Interpretor& di, Standard_Integer argc, con
//======================================================================= //=======================================================================
static Standard_Integer OCC902(Draw_Interpretor& di, Standard_Integer argc, const char ** argv) static Standard_Integer OCC902(Draw_Interpretor& di, Standard_Integer argc, const char ** argv)
{ {
if (argc =! 2) if (argc != 2)
{ {
di <<"Usage : " << argv[0] << " expression"<<"\n"; di <<"Usage : " << argv[0] << " expression"<<"\n";
return 1; return 1;

4
src/QANewBRepNaming/QANewBRepNaming_BooleanOperationFeat.cxx
View File

@ -1103,7 +1103,7 @@ static void SortEdges5 (const TopTools_Array1OfShape& theArS, const TColgp_Array
else {aDP = aDP3; aCP = theArP.Value(i4);} //i4, i5 - group of 2 edges at the bottom else {aDP = aDP3; aCP = theArP.Value(i4);} //i4, i5 - group of 2 edges at the bottom
if(!IsDirectionPositive(theAx, aDP, aCP)) {//first must be positive direction if(!IsDirectionPositive(theAx, aDP, aCP)) {//first must be positive direction
Standard_Integer aN; Standard_Integer aN;
if(aTop = 2) { if(aTop == 2) {
// change i1 <=> i2 // change i1 <=> i2
aN = i2; i2 = i1; aN = i2; i2 = i1;
i1 = aN; i1 = aN;
@ -1126,7 +1126,7 @@ static void SortEdges5 (const TopTools_Array1OfShape& theArS, const TColgp_Array
if(!IsDirectionPositive(theAx, aDP, aCP)) {//first must be positive direction if(!IsDirectionPositive(theAx, aDP, aCP)) {//first must be positive direction
Standard_Integer aN; Standard_Integer aN;
if(aTop = 2) { if(aTop == 2) {
// change i3 <=> i5 // change i3 <=> i5
aN = i5; i5 = i3; aN = i5; i5 = i3;
i3 = aN; i3 = aN;