Improve documentation

src/binres/binres.hpp

@@ -6,7 +6,6 @@
  * @date 2023-05-12
  * 
  * @copyright Copyright (c) 2023
- * 
  */
 #pragma once
 
@@ -19,6 +18,7 @@
  * @param icon The icon name
  * @param size Scale the icon to the specified size
  * @return wxBitmap 
+ * @ingroup binres
  */
 wxBitmap LoadPNG(std::string icon, wxSize size);
 
@@ -27,6 +27,7 @@ wxBitmap LoadPNG(std::string icon, wxSize size);
  * 
  * @param icon 
  * @return wxBitmap the image or wxNullBitmap if not found
+ * @ingroup binres
  */
 wxBitmap LoadPNG(std::string icon);
 

src/game_tab/HalfMove.hpp

@@ -27,11 +27,30 @@ public:
   HalfMove(std::string move_absolute,std::string move_san, std::string fen);
   HalfMove(CMI::HalfMove *m);
 
+  /**
+   * @brief Add mainline to the current move.
+   * If a mainline already exists, add to its variations
+   * 
+   * @param m the move to add
+   */
   void AddMove(HalfMove *m);
   /// @brief Check if current half move is within a variation
   bool IsVariation();
-  /// @brief Get the root of a variation
+  /**
+   * @brief Check if pointer @a m to a HalfMove is a parent of the current one
+   * 
+   * @param m 
+   * @return true 
+   * @return false 
+   */
   bool HasParent(HalfMove*m);
+  /**
+   * @brief Check if a given pointer @a m to a HalfMove is in mainline or variations of the current move.
+   * 
+   * @param m 
+   * @return true 
+   * @return false 
+   */
   bool HasChild(HalfMove*m);
   /// @brief Retrieve the list of moves from the current one to the first one
   std::vector<HalfMove *> GetLine();

src/game_tab/right_panel/LiveEngineDialog.hpp

@@ -26,15 +26,29 @@ class LiveEngineDialog : public DialogLiveEngine {
   wxTimer timer;
   /// @brief The following time interval definitely need to be configure in the user settings (set to 1s for now)
   std::uint32_t interval;
-
+  /**
+   * @brief Called to fetch last evaluation from the engine subprocess (stockfish, fritz etc.)
+   * 
+   * @param event 
+   */
+  void OnTimerTick(wxTimerEvent &event);
+  void OnClose(wxCloseEvent &e);
+  /**
+   * @brief Create the engine sub process using the uciadapter library
+   * 
+   */
+  void InitEngine();
+  /// @brief Pause/Resume evaluation
+  void TogglePauseEngine(wxCommandEvent &event);
 public:
   LiveEngineDialog(wxWindow *parent, std::uint32_t engine_id);
   ~LiveEngineDialog();
-  void InitEngine();
-  void TogglePauseEngine(wxCommandEvent &event);
-  void OnTimerTick(wxTimerEvent &event);
+  /**
+   * @brief Used setup a new position to evaluate
+   * 
+   * @param fen position to evaluate
+   */
   void SetFEN(std::string fen);
   void StopEngine();
   void StartEngine();
-  void OnClose(wxCloseEvent &e);
 };