Skip to Content.
Sympa Menu

ndt-dev - [ndt-dev] Issue 98 in ndt: Same code for IF and ELSE branches in code printing test results.

Subject: NDT-DEV email list created

List archive

[ndt-dev] Issue 98 in ndt: Same code for IF and ELSE branches in code printing test results.


Chronological Thread 
  • From:
  • To:
  • Subject: [ndt-dev] Issue 98 in ndt: Same code for IF and ELSE branches in code printing test results.
  • Date: Thu, 21 Nov 2013 12:23:06 +0000

Status: New
Owner: ----
Labels: Type-Defect Severity-Medium

New issue 98 by : Same code for IF and ELSE branches in code printing test results.
http://code.google.com/p/ndt/issues/detail?id=98

https://code.google.com/p/ndt/source/browse/trunk/Applet/Tcpbw100.java#3266

Both IF and ELSE branches have the same code.
AFAICT from other parts of this file with similar code (for example https://code.google.com/p/ndt/source/browse/trunk/Applet/Tcpbw100.java#3022), the desired code is probably supposed to only have the IF branch. I.e.,

if (_dC2sspd > _dSc2sspd) {
if (_dSc2sspd < (_dC2sspd * (1.0 - NDTConstants.VIEW_DIFF))) {
_txtStatistics.append(_resBundDisplayMsgs
.getString("c2s")
+ " "
+ _resBundDisplayMsgs.getString("qSeen")
+ ": "
+ NDTUtils.prtdbl(NDTConstants.PERCENTAGE * (_dC2sspd - _dSc2sspd)
/ _dC2sspd) + "%\n");
}
}

Same problem at line:
https://code.google.com/p/ndt/source/browse/trunk/Applet/Tcpbw100.java#3291

--
You received this message because this project is configured to send all issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings



Archive powered by MHonArc 2.6.16.

Top of Page