Remove unnecessary comparison operator overload

While normally harmless, it causes ambiguity if we use string mapping
that does not derive from std::string (e.g., to work around VC++ DLL
export mess).

1 files changed, 7 insertions, 0 deletions

diff --git a/libxsd/xsd/cxx/tree/elements.hxx b/libxsd/xsd/cxx/tree/elements.hxx
index b36f4fd..8a6c433 100644
--- a/libxsd/xsd/cxx/tree/elements.hxx
+++ b/libxsd/xsd/cxx/tree/elements.hxx
@@ -1838,11 +1838,18 @@ namespace xsd
           return s < table_[i];
         }
 
+        // This overload shouldn't be necessary according to the standard
+        // and removing it doesn't appear to trip any old compilers that
+        // we still support. But let's keeps preprocessed-out until we
+        // go C++11-only.
+        //
+#if 0
         bool
         operator() (std::size_t i, std::size_t j) const
         {
           return std::basic_string<C> (table_[i]) < table_[j];
         }
+#endif
 
       private:
         const C* const* table_;