Reorder includes is some simulator files to fix issues on msys2 (no code change)

Due to pegtl.hpp included in some files, reordering include fix some conflicts:
wxWidgets files must be included before pegtl.hpp, to compile with wxWidgets 3.0
I sometimes use wxWidgets 3.0 for debugging purposes.

eeschema/sim/ngspice.h

@@ -27,12 +27,14 @@
 #ifndef NGSPICE_H
 #define NGSPICE_H
 
-#include <sim/spice_simulator.h>
+#include <wx/dynlib.h>
+
+// Include simulator headers after wxWidgets headers to avoid conflicts with Windows headers
+// (especially on msys2)
+#include "sim/spice_simulator.h"
 #include <sim/sim_model.h>
 #include <sim/sim_value.h>
 
-#include <wx/dynlib.h>
-
 #include <ngspice/sharedspice.h>
 
 #include <enum_vector.h>

eeschema/sim/sim_lib_mgr.cpp

@@ -22,13 +22,16 @@
  * 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA
  */
 
-#include <sim/sim_lib_mgr.h>
 #include <sch_symbol.h>
-#include <sim/sim_library.h>
-#include <sim/sim_model.h>
 #include <pgm_base.h>
 #include <string>
 
+// Include simulator headers after wxWidgets headers to avoid conflicts with Windows headers
+// (especially on msys2)
+#include <sim/sim_library.h>
+#include <sim/sim_model.h>
+#include <sim/sim_lib_mgr.h>
+
 
 SIM_LIB_MGR::SIM_LIB_MGR( const PROJECT& aPrj ) : m_project( aPrj )
 {

eeschema/sim/sim_model.cpp

@@ -22,6 +22,9 @@
  * 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA
  */
 
+#include <lib_symbol.h>
+#include <confirm.h>
+
 #include <sim/sim_model.h>
 #include <sim/sim_model_behavioral.h>
 #include <sim/sim_model_ideal.h>
@@ -38,9 +41,6 @@
 
 #include <sim/sim_library_kibis.h>
 
-#include <lib_symbol.h>
-#include <confirm.h>
-
 #include <boost/algorithm/string/case_conv.hpp>
 #include <boost/algorithm/string/trim.hpp>
 #include <boost/algorithm/string.hpp>
@@ -833,7 +833,7 @@ std::unique_ptr<SIM_MODEL> SIM_MODEL::Create( TYPE aType )
     case TYPE::C:
     case TYPE::L:
         return std::make_unique<SIM_MODEL_IDEAL>( aType );
-    
+
     case TYPE::R_POT:
         return std::make_unique<SIM_MODEL_R_POT>();
 
@@ -924,7 +924,7 @@ SIM_MODEL::SIM_MODEL( TYPE aType,
 
 SIM_MODEL::SIM_MODEL( TYPE aType,
                       std::unique_ptr<SPICE_GENERATOR> aSpiceGenerator,
-                      std::unique_ptr<SIM_SERDE> aSerde ) : 
+                      std::unique_ptr<SIM_SERDE> aSerde ) :
         m_baseModel( nullptr ),
         m_serde( std::move( aSerde ) ),
         m_spiceGenerator( std::move( aSpiceGenerator ) ),

eeschema/sim/sim_model_kibis.cpp

@@ -21,14 +21,15 @@
  * 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA
  */
 
-#include <sim/kibis/kibis.h>
-#include <sim/sim_model_kibis.h>
-#include <sim/sim_library_kibis.h>
 #include <paths.h>
 #include <fmt/core.h>
 #include <wx/filename.h>
 #include <kiway.h>
 
+#include <sim/kibis/kibis.h>
+#include <sim/sim_model_kibis.h>
+#include <sim/sim_library_kibis.h>
+
 std::string SPICE_GENERATOR_KIBIS::ModelName( const SPICE_ITEM& aItem ) const
 {
     return fmt::format( "{}.{}", aItem.refName, aItem.baseModelName );

eeschema/sim/sim_model_spice.cpp

@@ -21,12 +21,14 @@
  * or you may write to the Free Software Foundation, Inc.,
  * 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA
  */
+#include <confirm.h>
 
+// Include simulator headers after wxWidgets headers to avoid conflicts with Windows headers
+// (especially on msys2)
 #include <sim/sim_model_spice.h>
 #include <sim/sim_model_raw_spice.h>
 #include <sim/spice_model_parser.h>
 #include <sim/sim_library_spice.h>
-#include <confirm.h>
 
 #include <boost/algorithm/string/trim.hpp>
 

eeschema/sim/sim_plot_frame.cpp

@@ -26,6 +26,9 @@
 
 #include <wx/debug.h>
 #include <wx/stc/stc.h>
+// For some obscure reason, needed on msys2 with some wxWidgets versions (3.0) to avoid
+// undefined symbol at link stage (due to use of #include <pegtl.hpp>)
+#include <wx/menu.h>
 
 #include <project/project_file.h>
 #include <sch_edit_frame.h>
@@ -40,12 +43,6 @@
 #include "string_utils.h"
 #include "ngspice_helpers.h"
 #include <pgm_base.h>
-#include "ngspice.h"
-#include "sim_plot_colors.h"
-#include "sim_plot_frame.h"
-#include "sim_plot_panel.h"
-#include "spice_simulator.h"
-#include "spice_reporter.h"
 #include <menus_helpers.h>
 #include <tool/tool_manager.h>
 #include <tools/ee_actions.h>
@@ -55,6 +52,14 @@
 #include <dialog_shim.h>
 #include <wx_filename.h>
 
+// Include simulator headers after wxWidgets headers to avoid conflicts with Windows headers
+// (especially on msys2)
+#include "ngspice.h"
+#include "sim_plot_colors.h"
+#include "sim_plot_frame.h"
+#include "sim_plot_panel.h"
+#include "spice_simulator.h"
+#include "spice_reporter.h"
 
 SIM_PLOT_TYPE operator|( SIM_PLOT_TYPE aFirst, SIM_PLOT_TYPE aSecond )
 {

eeschema/sim/sim_property.cpp

@@ -21,15 +21,17 @@
  * or you may write to the Free Software Foundation, Inc.,
  * 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA
  */
-
-#include <sim/sim_property.h>
-#include <sim/sim_value.h>
 #include <ki_exception.h>
 #include <confirm.h>
 #include <wx/combo.h>
 #include <wx/combobox.h>
 #include <wx/notebook.h>
 
+// Include simulator headers after wxWidgets headers to avoid conflicts with Windows headers
+// (especially on msys2)
+#include <sim/sim_property.h>
+#include <sim/sim_value.h>
+
 
 wxBEGIN_EVENT_TABLE( SIM_VALIDATOR, wxValidator )
     EVT_KEY_DOWN( SIM_VALIDATOR::onKeyDown )

eeschema/sim/sim_property.h

@@ -25,13 +25,15 @@
 #ifndef SIM_PROPERTY_H
 #define SIM_PROPERTY_H
 
-#include <sim/sim_model.h>
 #include <wx/window.h>
 #include <wx/notebook.h>
 #include <wx/propgrid/propgrid.h>
 #include <wx/propgrid/manager.h>
 #include <wx/propgrid/props.h>
 
+// Include this header after wxWidgets headers to avoid conflicts with Windows headers
+// (especially on msys2)
+#include <sim/sim_model.h>
 
 class SIM_VALIDATOR : public wxValidator
 {

eeschema/sim/sim_value.h

@@ -29,8 +29,10 @@
 #include <optional>
 #include <complex>
 #include <memory>
-#include <pegtl.hpp>
 
+// Include this header after wxWidgets headers to avoid conflicts with Windows headers
+// (especially on msys2)
+#include <pegtl.hpp>
 
 namespace SIM_VALUE_GRAMMAR
 {
@@ -161,7 +163,7 @@ namespace SIM_VALUE_GRAMMAR
 
     template <SIM_VALUE::TYPE ValueType>
     struct significand;
-    
+
     template <> struct significand<SIM_VALUE::TYPE_FLOAT> :
         sor<seq<intPart, one<'.'>, fracPart>,
             seq<intPart, one<'.'>>,

eeschema/sim/spice_simulator.cpp

@@ -23,10 +23,12 @@
  * 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA
  */
 
-#include "ngspice.h"
-
 #include <confirm.h>
 
+// Include simulator headers after wxWidgets headers to avoid conflicts with Windows headers
+// (especially on msys2)
+#include "ngspice.h"
+
 std::shared_ptr<SPICE_SIMULATOR> SIMULATOR::CreateInstance( const std::string& )
 {
     try

eeschema/sim/spice_value.cpp

@@ -22,11 +22,10 @@
  * 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA
  */
 
-#include "spice_value.h"
-
 #include <stdexcept>
 #include <cmath>
 
+#include <wx/event.h>
 #include <wx/textentry.h>
 #include <wx/numformatter.h>
 #include <confirm.h>
@@ -34,6 +33,11 @@
 #include <ki_exception.h>
 #include <locale_io.h>
 
+// Include simulator headers after wxWidgets headers to avoid conflicts with Windows headers
+// (especially on msys2)
+#include "spice_value.h"
+
+
 SPICE_VALUE::SPICE_VALUE( const wxString& aString )
 {
     char buf[8] = { 0, };