"Fixed" trouble with cursor that "already in use somewhere"

Riketta · Sep 4, 2018 · bee1c56 · bee1c56
1 parent 1cea2ed
commit bee1c56
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 3 deletions.
diff --git a/Warcraft Fishman/Bot.cs b/Warcraft Fishman/Bot.cs
@@ -68,8 +68,15 @@ public void FishingLoop()
                     action.Invoke(handle);
 
                 logger.Info("Starting fishing");
-                while (!Fishing(preset.GetActions(Action.Event.Fish)[0]))
-                    logger.Warn("Starting fishing again");
+                try
+                {
+                    while (!Fishing(preset.GetActions(Action.Event.Fish)[0]))
+                        logger.Warn("Starting fishing again");
+                }
+                catch (Exception ex)
+                {
+                    logger.Error("Exception occured while fishing: " + ex.ToString());
+                }
 
                 logger.Info("Invoking post-fishing actions");
                 foreach (var action in preset.GetActions(Action.Event.PostFish))

diff --git a/Warcraft Fishman/DeviceManager.cs b/Warcraft Fishman/DeviceManager.cs
@@ -39,7 +39,7 @@ public static Bitmap GetCurrentIcon()
                 {
                     using (Graphics g = Graphics.FromImage(icon))
                         WinApi.DrawIcon(g.GetHdc(), 0, 0, pci.hCursor);
-                    cursorIcon = new Bitmap(icon); // We have to "clone" icon because handle won't be released if we continue use it
+                    cursorIcon = new Bitmap(icon).Clone() as Bitmap; // We have to "clone" icon because handle won't be released if we continue use it
                 }
             }