[eve7] Include json_fwd.hpp if available

hahnjo · hahnjo · commit ed56a35f395a · 2022-08-19T13:52:31.000+02:00
Version 3.11 of nlohmann/json introduced "versioned, ABI-tagged inline namespace"s, which breaks our forward declaration. Fortunately, we can assume the json_fwd.hpp header to be present starting from that same version because the JSON_MultipleHeaders option now defaults to ON and even if not, json_fwd.hpp is installed since patch version 3.11.2. For earlier versions, both methods work but json_fwd.hpp isn't guaranteed to be installed. Still use it if available. Fixes root-project#11130
diff --git a/graf3d/eve7/inc/ROOT/REveElement.hxx b/graf3d/eve7/inc/ROOT/REveElement.hxx
@@ -16,15 +16,22 @@
 #include <ROOT/REveVector.hxx>
 #include <ROOT/REveProjectionBases.hxx>
 
-#include <map>
 #include <memory>
 #include <list>
 #include <ostream>
 #include <set>
 #include <string>
-#include <vector>
 
-class TGeoMatrix;
+#if __has_include(<nlohmann/json_fwd.hpp>)
+#include <nlohmann/json_fwd.hpp>
+#else
+// This forward declaration works for versions before 3.11.0, which introduced
+// a "versioned, ABI-tagged inline namespace". The json_fwd.hpp header is
+// present in multi-header installations (the default as of version 3.11.0), and
+// single-header installations of version 3.11.2+.
+
+#include <map>
+#include <vector>
 
 namespace nlohmann {
 template<typename T, typename SFINAE>
@@ -41,6 +48,10 @@ using json = basic_json<std::map, std::vector, std::string, bool, std::int64_t,
                         adl_serializer, std::vector<std::uint8_t>>;
 } // namespace nlohmann
 
+#endif
+
+class TGeoMatrix;
+
 namespace ROOT {
 namespace Experimental {
 
