From d4937c8166c0ffa4a4606aafb9a6fdbde76afc60 Mon Sep 17 00:00:00 2001
From: Niko Ehrenfeuchter <nikolaus.ehrenfeuchter@unibas.ch>
Date: Thu, 1 Mar 2018 15:06:51 +0100
Subject: [PATCH] Disable call to RoboCommand.Stop() method.

This commit is successfully tested to work on Windows 7 with the latest
version of RoboSharp (at the time of writing, including the
[EnableRaisingEvents fix][1] which has not yet been merged upstream).

This closes #15

[1]: https://github.com/tjscience/RoboSharp/pull/46
---
 ATxService/RoboCommand.cs | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/ATxService/RoboCommand.cs b/ATxService/RoboCommand.cs
index a220cd2..2bbb6a9 100644
--- a/ATxService/RoboCommand.cs
+++ b/ATxService/RoboCommand.cs
@@ -154,7 +154,12 @@ namespace ATxService
             if (_transferState == TxState.DoNothing)
                 return;
 
-            _roboCommand.Stop();
+
+            // Calling _roboCommand.Stop() is not required (any more, maybe it never was), since
+            // all of this is already taken care of by the RoboCommand class internally, hence we
+            // can safely disable the call here:
+            // _roboCommand.Stop();
+
             Log.Debug("Transfer stopped");
             _transferState = TxState.Stopped;
             _status.TransferredBytesCompleted = 0;
-- 
GitLab