From cdc0e012e902a3573f28351c8d87793ec80fe021 Mon Sep 17 00:00:00 2001
From: Niko Ehrenfeuchter <nikolaus.ehrenfeuchter@unibas.ch>
Date: Mon, 5 Feb 2018 08:17:41 +0100
Subject: [PATCH] Simplify calculation of transfer progress percentage.

Refers to #2, #26
---
 ATxService/RoboCommand.cs | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/ATxService/RoboCommand.cs b/ATxService/RoboCommand.cs
index df740f1..0a17e29 100644
--- a/ATxService/RoboCommand.cs
+++ b/ATxService/RoboCommand.cs
@@ -181,10 +181,9 @@ namespace ATxService
 
             _txCurFileProgress = progress;
             _status.TransferredBytesCurrentFile = (long) (_txCurFileSize * e.CurrentFileProgress / 100);
-            // NOTE: the (double) is required to make the division work on float which can then 
-            // eventually be cast into an (int) after multiplying it by 100:
-            _status.CurrentTransferPercent = (int)((_status.TransferredBytesCompleted + _status.TransferredBytesCurrentFile) * 100 /
-                                   _status.CurrentTransferSize);
+            _status.CurrentTransferPercent = (int) (
+                (_status.TransferredBytesCompleted + _status.TransferredBytesCurrentFile) * 100 /
+                _status.CurrentTransferSize);
             Log.Info("Current transfer at {0}%", _status.CurrentTransferPercent);
             Log.Trace("Tx progress: complete [{0}] - current [{1}] - combined {2:0}%",
                 _status.TransferredBytesCompleted, _status.TransferredBytesCurrentFile,
-- 
GitLab