Do analysis.set_finished() last

niklasf · niklasf · commit 984bf871d0fb · 2019-05-04T15:39:58.000+02:00
diff --git a/chess/engine.py b/chess/engine.py
@@ -1286,8 +1286,8 @@ def _info(self, engine: UciProtocol, arg: str) -> None:
             def _bestmove(self, engine: UciProtocol, arg: str) -> None:
                 if not self.result.done():
                     raise EngineError("was not searching, but engine sent bestmove")
-                self.analysis.set_finished()
                 self.set_finished()
+                self.analysis.set_finished()
 
             def cancel(self, engine: UciProtocol) -> None:
                 engine.send_line("stop")
@@ -1857,8 +1857,8 @@ def end(self, engine: XBoardProtocol) -> None:
                 if self.time_limit_handle:
                     self.time_limit_handle.cancel()
 
-                self.analysis.set_finished()
                 self.set_finished()
+                self.analysis.set_finished()
 
             def cancel(self, engine: XBoardProtocol) -> None:
                 if self.stopped:
