Fix comment and remove useless inner type definition

src/mesh/SubItemArrayPerItem.hpp

@@ -42,15 +42,13 @@ class SubItemArrayPerItem
   // Allow const std:weak_ptr version to access our data
   friend SubItemArrayPerItem<std::add_const_t<DataType>, ItemOfItem, ConnectivityWeakPtr>;
 
-  // Allow const std:shared_ptr version to access our data
+  // Allow non-const std:shared_ptr version to access our data
   friend SubItemArrayPerItem<std::remove_const_t<DataType>, ItemOfItem, ConnectivitySharedPtr>;
 
-  // Allow const std:weak_ptr version to access our data
+  // Allow non-const std:weak_ptr version to access our data
   friend SubItemArrayPerItem<std::remove_const_t<DataType>, ItemOfItem, ConnectivityWeakPtr>;
 
  public:
-  using ToShared = SubItemArrayPerItem<DataType, ItemOfItem, ConnectivitySharedPtr>;
-
   friend PUGS_INLINE SubItemArrayPerItem<std::remove_const_t<DataType>, ItemOfItem, ConnectivityPtr>
   copy(SubItemArrayPerItem<DataType, ItemOfItem, ConnectivityPtr>& source)
   {

src/mesh/SubItemValuePerItem.hpp

@@ -41,15 +41,13 @@ class SubItemValuePerItem
   // Allow const std:weak_ptr version to access our data
   friend SubItemValuePerItem<std::add_const_t<DataType>, ItemOfItem, ConnectivityWeakPtr>;
 
-  // Allow const std:shared_ptr version to access our data
+  // Allow non-const std:shared_ptr version to access our data
   friend SubItemValuePerItem<std::remove_const_t<DataType>, ItemOfItem, ConnectivitySharedPtr>;
 
-  // Allow const std:weak_ptr version to access our data
+  // Allow non-const std:weak_ptr version to access our data
   friend SubItemValuePerItem<std::remove_const_t<DataType>, ItemOfItem, ConnectivityWeakPtr>;
 
  public:
-  using ToShared = SubItemValuePerItem<DataType, ItemOfItem, ConnectivitySharedPtr>;
-
   friend PUGS_INLINE SubItemValuePerItem<std::remove_const_t<DataType>, ItemOfItem, ConnectivityPtr>
   copy(SubItemValuePerItem<DataType, ItemOfItem, ConnectivityPtr>& source)
   {