[thirdparty] Set IMGUI_DEFINE_MATH_OPERATORS (#7899)

This avoids the need to define it early in the include order everywhere else.

Signed-off-by: Jade Turner <spacey-sooty@proton.me>

cscore/.styleguide

@@ -38,8 +38,10 @@ includeOtherLibs {
   ^fmt/
   ^gtest/
   ^opencv2/
+  ^imgui
   ^support/
   ^tcpsockets/
   ^wpi/
+  ^wpigui
   ^wpinet/
 }

cscore/examples/usbviewer/usbviewer.cpp

@@ -6,7 +6,6 @@
 #include <thread>
 #include <vector>
 
-#define IMGUI_DEFINE_MATH_OPERATORS
 #include <imgui.h>
 #include <imgui_internal.h>
 #include <opencv2/core/core.hpp>

datalogtool/src/main/native/cpp/App.cpp

@@ -10,11 +10,9 @@
 #include <glass/Context.h>
 #include <glass/MainMenuBar.h>
 #include <glass/Storage.h>
-#include <libssh/libssh.h>
-
-#define IMGUI_DEFINE_MATH_OPERATORS
 #include <imgui.h>
 #include <imgui_internal.h>
+#include <libssh/libssh.h>
 #include <wpigui.h>
 #include <wpigui_openurl.h>
 

datalogtool/src/main/native/cpp/App.h

@@ -4,7 +4,6 @@
 
 #pragma once
 
-#define IMGUI_DEFINE_MATH_OPERATORS
 #include <imgui.h>
 
 void SetNextWindowPos(const ImVec2& pos, ImGuiCond cond = 0,

glass/.styleguide

@@ -26,8 +26,10 @@ includeOtherLibs {
   ^frc/
   ^google/
   ^imgui
+  portable-file-dialogs.h
   ^networktables/
   ^ntcore
+  ^units
   ^wpi/
   ^wpigui
   ^wpimath/

glass/src/lib/native/cpp/hardware/Gyro.cpp

@@ -7,8 +7,6 @@
 #include <cmath>
 #include <numbers>
 
-#define IMGUI_DEFINE_MATH_OPERATORS
-
 #include <imgui.h>
 #include <imgui_internal.h>
 #include <wpi/StringExtras.h>

glass/src/lib/native/cpp/other/Drive.cpp

@@ -6,12 +6,10 @@
 
 #include <array>
 #include <cmath>
-
-#define IMGUI_DEFINE_MATH_OPERATORS
+#include <numbers>
 
 #include <imgui.h>
 #include <imgui_internal.h>
-#include <numbers>
 
 #include "glass/Context.h"
 #include "glass/DataSource.h"

glass/src/lib/native/cpp/other/Field2D.cpp

@@ -17,8 +17,6 @@
 #include <frc/geometry/Pose2d.h>
 #include <frc/geometry/Rotation2d.h>
 #include <frc/geometry/Translation2d.h>
-
-#define IMGUI_DEFINE_MATH_OPERATORS
 #include <imgui.h>
 #include <imgui_internal.h>
 #include <imgui_stdlib.h>

glass/src/lib/native/cpp/other/Mechanism2D.cpp

@@ -17,8 +17,6 @@
 #include <frc/geometry/Rotation2d.h>
 #include <frc/geometry/Transform2d.h>
 #include <frc/geometry/Translation2d.h>
-
-#define IMGUI_DEFINE_MATH_OPERATORS
 #include <imgui.h>
 #include <imgui_internal.h>
 #include <imgui_stdlib.h>

glass/src/lib/native/cpp/other/Plot.cpp

@@ -22,7 +22,6 @@
 #pragma GCC diagnostic ignored "-Wformat-nonliteral"
 #endif
 
-#define IMGUI_DEFINE_MATH_OPERATORS
 #include <imgui.h>
 #include <imgui_stdlib.h>
 #include <implot.h>

glass/src/lib/native/cpp/support/ExtraGuiWidgets.cpp

@@ -4,14 +4,12 @@
 
 #include "glass/support/ExtraGuiWidgets.h"
 
-#define IMGUI_DEFINE_MATH_OPERATORS
-#include <imgui.h>
-#include <imgui_internal.h>
-
 #include <cstdio>
 #include <cstdlib>
 #include <cstring>
 
+#include <imgui.h>
+#include <imgui_internal.h>
 #include <wpi/DenseMap.h>
 
 #include "glass/DataSource.h"

glass/src/lib/native/include/glass/MainMenuBar.h

@@ -4,12 +4,12 @@
 
 #pragma once
 
-#include <portable-file-dialogs.h>
-
 #include <functional>
 #include <memory>
 #include <vector>
 
+#include <portable-file-dialogs.h>
+
 namespace glass {
 
 /**

glass/src/lib/native/include/glass/Window.h

@@ -9,7 +9,6 @@
 #include <string_view>
 #include <utility>
 
-#define IMGUI_DEFINE_MATH_OPERATORS
 #include <imgui.h>
 
 #include "glass/View.h"

glass/src/lib/native/include/glass/other/Field2D.h

@@ -10,8 +10,6 @@
 #include <frc/geometry/Pose2d.h>
 #include <frc/geometry/Rotation2d.h>
 #include <frc/geometry/Translation2d.h>
-
-#define IMGUI_DEFINE_MATH_OPERATORS
 #include <imgui.h>
 #include <wpi/function_ref.h>
 

glass/src/lib/native/include/glass/other/Mechanism2D.h

@@ -6,8 +6,6 @@
 
 #include <frc/geometry/Rotation2d.h>
 #include <frc/geometry/Translation2d.h>
-
-#define IMGUI_DEFINE_MATH_OPERATORS
 #include <imgui.h>
 #include <wpi/function_ref.h>
 

glass/src/lib/native/include/glass/support/ExtraGuiWidgets.h

@@ -6,7 +6,6 @@
 
 #include <stdint.h>
 
-#define IMGUI_DEFINE_MATH_OPERATORS
 #include <imgui.h>
 
 namespace glass {

thirdparty/imgui_suite/imgui/include/imconfig.h

@@ -98,7 +98,7 @@
         operator MyVec4() const { return MyVec4(x,y,z,w); }
 */
 //---- ...Or use Dear ImGui's own very basic math operators.
-//#define IMGUI_DEFINE_MATH_OPERATORS
+#define IMGUI_DEFINE_MATH_OPERATORS
 
 //---- Use 32-bit vertex indices (default is 16-bit) is one way to allow large meshes with more than 64K vertices.
 // Your renderer backend will need to support it (most example renderer backends support both 16/32-bit indices).

upstream_utils/imgui_patches/0001-Set-IMGUI_DEFINE_MATH_OPERATORS.patch

@@ -0,0 +1,23 @@
+From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
+From: Jade Turner <spacey-sooty@proton.me>
+Date: Fri, 11 Apr 2025 11:20:41 +0800
+Subject: [PATCH] Set IMGUI_DEFINE_MATH_OPERATORS
+
+Signed-off-by: Jade Turner <spacey-sooty@proton.me>
+---
+ imconfig.h | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/imconfig.h b/imconfig.h
+index b56ba49467ebb388e038f0a23aed46044085bdff..291ce1995412f10e784beaf3f4c51e02d90b9813 100644
+--- a/imconfig.h
++++ b/imconfig.h
+@@ -98,7 +98,7 @@
+         operator MyVec4() const { return MyVec4(x,y,z,w); }
+ */
+ //---- ...Or use Dear ImGui's own very basic math operators.
+-//#define IMGUI_DEFINE_MATH_OPERATORS
++#define IMGUI_DEFINE_MATH_OPERATORS
+ 
+ //---- Use 32-bit vertex indices (default is 16-bit) is one way to allow large meshes with more than 64K vertices.
+ // Your renderer backend will need to support it (most example renderer backends support both 16/32-bit indices).