Drop XML format again

JSON should be enough as human readable and easy to interchange format

g2o/core/CMakeLists.txt

@@ -44,7 +44,6 @@ io/io_format.cpp
 io/io_binary.cpp io/io_binary.h
 io/io_json.cpp io/io_json.h
 io/io_g2o.cpp io/io_g2o.h
-io/io_xml.cpp io/io_xml.h
 )
 
 target_include_directories(core PUBLIC

g2o/core/abstract_graph.cpp

@@ -32,7 +32,6 @@
 #include "g2o/core/io/io_binary.h"
 #include "g2o/core/io/io_format.h"
 #include "g2o/core/io/io_json.h"
-#include "g2o/core/io/io_xml.h"
 #include "g2o/stuff/logger.h"
 #include "io/io_g2o.h"
 
@@ -54,8 +53,6 @@ std::unique_ptr<g2o::IoInterface> allocate(g2o::io::Format format) {
       return std::make_unique<g2o::IoBinary>();
     case g2o::io::Format::kJson:
       return std::make_unique<g2o::IoJson>();
-    case g2o::io::Format::kXML:
-      return std::make_unique<g2o::IoXml>();
   }
   G2O_CRITICAL("Failed to create graph IO interface for format {}",
                g2o::io::to_string(format));

g2o/core/io/io_format.cpp

@@ -41,16 +41,13 @@ std::string_view to_string(g2o::io::Format format) {
       return "Binary";
     case g2o::io::Format::kJson:
       return "JSON";
-    case g2o::io::Format::kXML:
-      return "XML";
   }
   return "";
 }
 
 std::optional<Format> formatForFileExtension(std::string_view extension) {
   if (extension == "g2o" || extension == "G2O") return Format::kG2O;
   if (extension == "json" || extension == "JSON") return Format::kJson;
-  if (extension == "xml" || extension == "XML") return Format::kXML;
   if (extension == "bin" || extension == "BIN") return Format::kBinary;
   return std::nullopt;
 }
@@ -66,7 +63,6 @@ std::vector<FileFilter> getFileFilter(bool open) {
   std::vector<FileFilter> result;
   result.emplace_back("g2o Ascii files (*.g2o)", Format::kG2O);
   result.emplace_back("g2o Json files (*.json)", Format::kJson);
-  result.emplace_back("g2o XML files (*.xml)", Format::kXML);
   result.emplace_back("g2o BIN files (*.bin)", Format::kBinary);
   if (open) {
     result.emplace_back("All Files (*)", Format::kUndefined);

g2o/core/io/io_format.h

@@ -40,7 +40,6 @@ enum class G2O_CORE_API Format {
   kG2O = 0,
   kBinary = 1,
   kJson = 2,
-  kXML = 3
 };
 
 G2O_CORE_API std::string_view to_string(g2o::io::Format format);

python/core/py_io_format.cpp

@@ -8,8 +8,7 @@ void declareIOFormat(py::module& m) {
   py::enum_<io::Format>(m, "IoFormat")
       .value("G2O", io::Format::kG2O, "G2O's custom ASCII IO format")
       .value("JSON", io::Format::kJson, "JSON IO format")
-      .value("BINARY", io::Format::kBinary, "G2O's custom binary format")
-      .value("XML", io::Format::kXML, "XML IO format");
+      .value("BINARY", io::Format::kBinary, "G2O's custom binary format");
 }
 
 }  // end namespace g2o

unit_test/general/graph_io.cpp

@@ -304,13 +304,13 @@ TEST_P(OptimizableGraphIO, SaveAndLoad) {
 
 namespace {
 // We can always test G2O format, others depend on libraries
-const auto kFileformatsToTest = Values(
-    g2o::io::Format::kG2O
+const auto kFileformatsToTest =
+    Values(g2o::io::Format::kG2O
 #ifdef G2O_HAVE_CEREAL
-    ,
-    g2o::io::Format::kJson, g2o::io::Format::kXML, g2o::io::Format::kBinary
+           ,
+           g2o::io::Format::kJson, g2o::io::Format::kBinary
 #endif
-);
+    );
 }  // namespace
 
 INSTANTIATE_TEST_SUITE_P(AbstractGraph, AbstractGraphIO, kFileformatsToTest);
@@ -327,7 +327,7 @@ TEST(OptimizableGraphIO, FileFilter) {
 TEST(OptimizableGraphIO, FormatToString) {
   static constexpr g2o::io::Format kAllFormats[] = {
       g2o::io::Format::kUndefined, g2o::io::Format::kG2O,
-      g2o::io::Format::kBinary, g2o::io::Format::kJson, g2o::io::Format::kXML};
+      g2o::io::Format::kBinary, g2o::io::Format::kJson};
   for (const auto& f : kAllFormats)
     EXPECT_THAT(g2o::io::to_string(f), Not(IsEmpty()));
 

unit_test/general/graph_io_dynamic.cpp

@@ -180,7 +180,6 @@ const auto kFileformatsToTest =
 #ifdef G2O_HAVE_CEREAL
                ,
            std::make_tuple(g2o::io::Format::kJson, true),
-           std::make_tuple(g2o::io::Format::kXML, true),
            std::make_tuple(g2o::io::Format::kBinary, true)
 #endif
     );