review comments

Author: d-w-moore

irods/meta.py

@@ -134,7 +134,7 @@ class iRODSMetaCollection:
     def __getattr__(self, name):
         from irods.manager.metadata_manager import _MetadataManager_opts_initializer
         # Separating _MetadataManager_opts_initializer from the MetadataManager class
-        # prevents # the possibility of arbitrary access by copy.copy() to parts of
+        # prevents the possibility of arbitrary access by copy.copy() to parts of
         # our object's state before they have been initialized, as it is known to do
         # by calling hasattr on the "__setstate__" attribute. The result of such
         # unfettered access is infinite recursion.  See:

irods/test/meta_test.py

@@ -820,8 +820,6 @@ def test_binary_avu_fields__issue_707(self):
     def test_cascading_changes_of_metadata_manager_options__issue_709(self):
         d = None
 
-#       def get_option(metacoll, key):
-#           return metacoll._manager._opts[key]
         try:
             d = self.sess.data_objects.create(f'{self.coll.path}/issue_709_test_1')
             m = d.metadata