COMP: Guard Extent.hpp against Windows.h min/max macros

* Undef min and max at the top of Extent.hpp so MSVC's transitively
  included Windows.h macros do not expand against the struct's member
  names in the constructor initializer list
* Fixes cascading fmt/color.h namespace errors on v143 Windows builds
  caused by MSVC losing its parse state after the Extent syntax error
* Re-sort includes in AbstractDataStore.hpp to satisfy clang-format

Signed-off-by: Joey Kleingers <joey.kleingers@bluequartz.net>

Author: joeykleingers

src/simplnx/Common/Extent.hpp

@@ -8,6 +8,17 @@
 #include <stdexcept>
 #include <vector>
 
+// Windows.h (pulled in transitively on MSVC builds) defines min/max as function-like
+// macros, which collide with the public member names below. Undef them so the member
+// declarations parse correctly and so std::min/std::max inside the inline methods are
+// not expanded.
+#ifdef min
+#undef min
+#endif
+#ifdef max
+#undef max
+#endif
+
 namespace nx::core
 {
 struct Extent

src/simplnx/DataStructure/AbstractDataStore.hpp

@@ -1,7 +1,7 @@
 #pragma once
 
-#include "simplnx/Common/Result.hpp"
 #include "simplnx/Common/Extent.hpp"
+#include "simplnx/Common/Result.hpp"
 #include "simplnx/Common/StringLiteralFormatting.hpp"
 #include "simplnx/Common/TypesUtility.hpp"
 #include "simplnx/DataStructure/IDataStore.hpp"