Fix Windows compilation in image_publisher.cpp (backport #1061) (#1062)

PR #985 added some code that used `M_PI`, but `M_PI` is not defined in any standard, and before including `cmath` or `math.h` in Windows it is necessary to define `_USE_MATH_DEFINES` to ensure that `M_PI` is defined.<hr>This is an automatic backport of pull request #1061 done by [Mergify](https://mergify.com). Co-authored-by: Silvio Traversaro <[email protected]>
ros-perception · Jan 6, 2025 · c3baf0d · c3baf0d
1 parent 426af55
commit c3baf0d
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/image_publisher/src/image_publisher.cpp b/image_publisher/src/image_publisher.cpp
@@ -31,6 +31,10 @@
 // ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
 // POSSIBILITY OF SUCH DAMAGE.
 
+// This define is added to support M_PI on Windows
+#ifndef _USE_MATH_DEFINES
+#define _USE_MATH_DEFINES
+#endif
 #include <cmath>
 #include <chrono>
 #include <limits>