<div dir="ltr"><div><div><div><div><div><div>This is with regard to the bug listed on <a href="https://trac.osgeo.org/gdal/ticket/6388" target="_blank">https://trac.osgeo.org/gdal/ticket/6388</a><br></div><br>The fix is to remove the hard coded pi constants, but apparently, using M_PI from math.h as it is breaks compilation on Visual Studio <a href="http://stackoverflow.com/questions/26065359/m-pi-flagged-as-undeclared-identifier" target="_blank">http://stackoverflow.com/questions/26065359/m-pi-flagged-as-undeclared-identifier</a> (I haven't checked it yet myself; that would need me to switch to windows, and I'm on linux right now)<br><br></div>However, we could also declare #define _USE_MATH_CONSTANTS, and then use M_PI without declaring it(And not just M_PI, all the other math constants listed on <a href="https://msdn.microsoft.com/en-us/library/4hwaceh6.aspx">https://msdn.microsoft.com/en-us/library/4hwaceh6.aspx</a>).<br></div><br></div>Could someone please check whether math constants work in later versions of visual studio(Both with and without #define _USE_MATH_CONSTANTS)?<br><br></div>Thank you<br></div>Tanuj<br></div>