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
- [ndt-dev] Issue 98 in ndt: Same code for IF and ELSE branches in code printing test results., ndt, 11/21/2013
- [ndt-dev] Re: Issue 98 in ndt: Same code for IF and ELSE branches in code printing test results., ndt, 11/21/2013
- [ndt-dev] Re: Issue 98 in ndt: Same code for IF and ELSE branches in code printing test results., ndt, 11/21/2013
- [ndt-dev] Re: Issue 98 in ndt: Same code for IF and ELSE branches in code printing test results., ndt, 11/21/2013
- [ndt-dev] Re: Issue 98 in ndt: Same code for IF and ELSE branches in code printing test results., ndt, 11/26/2013
- [ndt-dev] Re: Issue 98 in ndt: Same code for IF and ELSE branches in code printing test results., ndt, 11/28/2013
Archive powered by MHonArc 2.6.16.