Opened 4 years ago

Closed 3 years ago

#72 closed defect (fixed)

Semicolon at the end of an if-statement looks like a bug.

Reported by: petter Owned by: somebody
Priority: major Milestone:
Component: component1 Version:
Keywords: Cc:

Description

The following code is from ClpSolver?.cpp (starting at line 7262 in my revision).

		  for (int i = 0; i < numberRows2; i++) {
		    if (sub[iBlock].getRowStatus(i)==basic && fabs(solution[i])<1.0e-7)
		      solution[i]=0.0;
		    if (solution[i] > dualTolerance_) {
		      // at upper
		      if (saveUpper[i] > 1.0e20)
			solution[i]=0.0;
		      objValue += solution[i] * saveUpper[i];
		    } else if (solution[i] < -dualTolerance_) {
		      // at lower
		      if (saveLower[i] < -1.0e20);
			solution[i]=0.0;
		      objValue += solution[i] * saveLower[i];
		    } else {
		      solution[i]=0.0;
		    }
		  }

The semicolon at the end of the if-statement looks like a bug to me.

Change History (2)

comment:1 Changed 3 years ago by ralacroix

Fixed in r2188 and released in CLP 1.16.10

comment:2 Changed 3 years ago by forrest

  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.