From 6eb87d89fd42bd57b1cae39a25f92e3afd73df36 Mon Sep 17 00:00:00 2001
From: Alexandra Heuschling <aheuschling@cox.informatik.rwth-aachen.de>
Date: Fri, 26 Feb 2021 19:13:09 +0100
Subject: [PATCH] clean up old code

---
 src/OpenMesh/Core/IO/reader/OMReader.cc | 207 +-----------------------
 src/OpenMesh/Core/IO/reader/OMReader.hh |   5 -
 2 files changed, 1 insertion(+), 211 deletions(-)

diff --git a/src/OpenMesh/Core/IO/reader/OMReader.cc b/src/OpenMesh/Core/IO/reader/OMReader.cc
index 68270a65..19800f69 100644
--- a/src/OpenMesh/Core/IO/reader/OMReader.cc
+++ b/src/OpenMesh/Core/IO/reader/OMReader.cc
@@ -806,7 +806,7 @@ size_t _OMReader_::restore_binary_custom_data(std::istream& _is, BaseProperty* _
 #endif
 
 #if defined(OM_DEBUG)
-      assert( block_size == b );
+      assert( block_size == b);
 #endif
 
       assert( block_size == _bp->size_of());
@@ -863,212 +863,7 @@ void _OMReader_:: add_generic_property(OMFormat::Chunk::PropertyName& _property_
   default:
     ;
   }
- 
-  
-//  if(_property_type == get_string_for_type(bool()))
-//  {
-//    add_generic_property_aux<bool>(_bi);
-//  }
-//  else if(_property_type == get_string_for_type(char()))
-//  {
-//    add_generic_property_aux<char>(_bi);
-//  }
-//  else if(_property_type == get_string_for_type(double()))
-//  {
-//    add_generic_property_aux<double>(_bi);
-//  }
-//  else if(_property_type == get_string_for_type(float()))
-//  {
-//    add_generic_property_aux<float>(_bi);
-//  }
-//  else if(_property_type == get_string_for_type(int()))
-//  {
-//    add_generic_property_aux<int>(_bi);
-//  }
-//  else if(_property_type == get_string_for_type(long()))
-//  {
-//    add_generic_property_aux<long>(_bi);
-//  }
-//  else if(_property_type == get_string_for_type(short()))
-//  {
-//    add_generic_property_aux<short>(_bi);
-//  }
-//  else if(_property_type == get_string_for_type(uchar()))
-//  {
-//   add_generic_property_aux<uchar>(_bi);
-//  }
-//  else if(_property_type == get_string_for_type(uint()))
-//  {
-//    add_generic_property_aux<uint>(_bi);
-//  }
-//  else if(_property_type == get_string_for_type(ulong()))
-//  {
-//    add_generic_property_aux<ulong>(_bi);
-//  }
-//  else
-//  {
-//    check_all_vector_types(_property_type, _bi);
-//  }
 }
-
-void _OMReader_::check_all_vector_types(OMFormat::Chunk::PropertyName& _property_type, BaseImporter& _bi) const
-{
-    check_all_vector_types_aux<1>(_property_type, _bi);
-    check_all_vector_types_aux<2>(_property_type, _bi);
-    check_all_vector_types_aux<3>(_property_type, _bi);
-    check_all_vector_types_aux<4>(_property_type, _bi);
-    check_all_std_vector_types_aux(_property_type, _bi);
-}
-
-void _OMReader_::check_all_std_vector_types_aux(OMFormat::Chunk::PropertyName& _property_type, BaseImporter& _bi) const
-{
-  if(_property_type == get_string_for_type(std::vector<bool>()))
-  {
-    add_generic_property_aux<std::vector<bool>>(_bi);
-  }
-  else if(_property_type == get_string_for_type(std::vector<char>()))
-  {
-    add_generic_property_aux<std::vector<char>>(_bi);
-  }
-  else if(_property_type == get_string_for_type(std::vector<signed char>()))
-  {
-    add_generic_property_aux<std::vector<signed char>>(_bi);
-  }
-  else if(_property_type == get_string_for_type(std::vector<double>()))
-  {
-    add_generic_property_aux<std::vector<double>>(_bi);
-  }
-  else if(_property_type == get_string_for_type(std::vector<float>()))
-  {
-    add_generic_property_aux<std::vector<float>>(_bi);
-  }
-  else if(_property_type == get_string_for_type(std::vector<int>()))
-  {
-    add_generic_property_aux<std::vector<int>>(_bi);
-  }
-  else if(_property_type == get_string_for_type(std::vector<long>()))
-  {
-    add_generic_property_aux<std::vector<long>>(_bi);
-  }
-  else if(_property_type == get_string_for_type(std::vector<short>()))
-  {
-    add_generic_property_aux<std::vector<short>>(_bi);
-  }
-  else if(_property_type == get_string_for_type(std::vector<uchar>()))
-  {
-   add_generic_property_aux<std::vector<uchar>>(_bi);
-  }
-  else if(_property_type == get_string_for_type(std::vector<uint>()))
-  {
-    add_generic_property_aux<std::vector<uint>>(_bi);
-  }
-  else if(_property_type == get_string_for_type(std::vector<ulong>()))
-  {
-    add_generic_property_aux<std::vector<ulong>>(_bi);
-  }
-  else if(_property_type == get_string_for_type(std::vector<ushort>()))
-  {
-    add_generic_property_aux<std::vector<ushort>>(_bi);
-  }
-}
-
-template<int Dim>
-void _OMReader_::check_all_vector_types_aux(OMFormat::Chunk::PropertyName& _property_type, BaseImporter& _bi) const
-{
-  if(_property_type == get_string_for_type(OpenMesh::VectorT<bool, Dim>()))
-  {
-    add_generic_property_aux<OpenMesh::VectorT<bool, Dim>>(_bi);
-  }
-  else if(_property_type == get_string_for_type(OpenMesh::VectorT<char, Dim>()))
-  {
-    add_generic_property_aux<OpenMesh::VectorT<char, Dim>>(_bi);
-  }
-  else if(_property_type == get_string_for_type(OpenMesh::VectorT<signed char, Dim>()))
-  {
-    add_generic_property_aux<OpenMesh::VectorT<signed char, Dim>>(_bi);
-  }
-  else if(_property_type == get_string_for_type(OpenMesh::VectorT<double, Dim>()))
-  {
-    add_generic_property_aux<OpenMesh::VectorT<double, Dim>>(_bi);
-  }
-  else if(_property_type == get_string_for_type(OpenMesh::VectorT<float, Dim>()))
-  {
-    add_generic_property_aux<OpenMesh::VectorT<float, Dim>>(_bi);
-  }
-  else if(_property_type == get_string_for_type(OpenMesh::VectorT<int, Dim>()))
-  {
-    add_generic_property_aux<OpenMesh::VectorT<int, Dim>>(_bi);
-  }
-  else if(_property_type == get_string_for_type(OpenMesh::VectorT<long, Dim>()))
-  {
-    add_generic_property_aux<OpenMesh::VectorT<long, Dim>>(_bi);
-  }
-  else if(_property_type == get_string_for_type(OpenMesh::VectorT<short, Dim>()))
-  {
-    add_generic_property_aux<OpenMesh::VectorT<short, Dim>>(_bi);
-  }
-  else if(_property_type == get_string_for_type(OpenMesh::VectorT<uchar, Dim>()))
-  {
-   add_generic_property_aux<OpenMesh::VectorT<uchar, Dim>>(_bi);
-  }
-  else if(_property_type == get_string_for_type(OpenMesh::VectorT<uint, Dim>()))
-  {
-    add_generic_property_aux<OpenMesh::VectorT<uint, Dim>>(_bi);
-  }
-  else if(_property_type == get_string_for_type(OpenMesh::VectorT<ulong, Dim>()))
-  {
-    add_generic_property_aux<OpenMesh::VectorT<ulong, Dim>>(_bi);
-  }
-  else if(_property_type == get_string_for_type(OpenMesh::VectorT<ushort, Dim>()))
-  {
-    add_generic_property_aux<OpenMesh::VectorT<ushort, Dim>>(_bi);
-  }
-}
-
-template<typename T>
-void _OMReader_::add_generic_property_aux(BaseImporter& _bi) const
-{
-  switch (chunk_header_.entity_)
-  {
-  case OMFormat::Chunk::Entity_Vertex:
-  {
-    VPropHandleT<T> prop;
-    _bi.kernel()->add_property(prop, property_name_);
-    break;
-  }
-  case OMFormat::Chunk::Entity_Face:
-  {
-    FPropHandleT<T> prop;
-    _bi.kernel()->add_property(prop, property_name_);
-    break;
-  }
-  case OMFormat::Chunk::Entity_Edge:
-  {
-    EPropHandleT<T> prop;
-    _bi.kernel()->add_property(prop, property_name_);
-    break;
-  }
-  case OMFormat::Chunk::Entity_Halfedge:
-  {
-    HPropHandleT<T> prop;
-    _bi.kernel()->add_property(prop, property_name_);
-    break;
-  }
-  case OMFormat::Chunk::Entity_Mesh:
-  {
-    MPropHandleT<T> prop;
-    _bi.kernel()->add_property(prop, property_name_);
-    break;
-  }
-  case OMFormat::Chunk::Entity_Sentinel:
-    ;
-  break;
-  default:
-    ;
-  }
-}
-
-
 //-----------------------------------------------------------------------------
 
 //=============================================================================
diff --git a/src/OpenMesh/Core/IO/reader/OMReader.hh b/src/OpenMesh/Core/IO/reader/OMReader.hh
index 74de1409..d29a8820 100644
--- a/src/OpenMesh/Core/IO/reader/OMReader.hh
+++ b/src/OpenMesh/Core/IO/reader/OMReader.hh
@@ -157,11 +157,6 @@ private:
   //------------------helper
 private:
   void add_generic_property(OMFormat::Chunk::PropertyName& _property_type, BaseImporter& _bi) const;
-  template <typename T> void add_generic_property_aux(BaseImporter& _bi) const;
-
-  void check_all_vector_types(OMFormat::Chunk::PropertyName& _property_type, BaseImporter& _bi) const;
-  template<int Dim> void check_all_vector_types_aux(OMFormat::Chunk::PropertyName& _property_type, BaseImporter& _bi) const;
-  void check_all_std_vector_types_aux(OMFormat::Chunk::PropertyName& _property_type, BaseImporter& _bi) const;
 };