Fix comments; cosmetic changes.

Author: pp-mo

lib/iris/fileformats/_nc_load_rules/engine.py

@@ -17,8 +17,6 @@
 API of a Pyke.knowlege_base, so that we can list its case-specific facts, as
 used in :meth:`iris.fileformats.netcdf._actions_activation_stats`.
 
-TODO: get rid of redundact fact-database identifier arguments.
-
 """
 from .actions import run_actions
 
@@ -98,6 +96,17 @@ def activate(self):
         """
         run_actions(self)
 
+    def get_kb(self):
+        """
+        Get a FactEntity, which mimic (bits of) a knowledge-base.
+
+        Just allowing
+        :meth:`iris.fileformats.netcdf._action_activation_stats` to list the
+        facts.
+
+        """
+        return self.facts
+
     def print_stats(self):
         """
         No-op, called by
@@ -110,32 +119,21 @@ def add_case_specific_fact(self, fact_name, fact_arglist):
         """
         Record a fact about the current output operation.
 
-        Roughly, self.facts.entity_lists[fact_name].append(fact_arglist).
-
-        NOTE: 'kb_name' is no longer used.
+        Roughly,
+          facts = self.facts.entity_lists[fact_name].case_specific_facts
+          facts.append(fact_arglist)
 
         """
         self.facts.add_fact(fact_name, fact_arglist)
 
-    def get_kb(self):
-        """
-        Get a FactEntity, which mimic (bits of) a knowledge-base.
-
-        Just allowing
-        :meth:`iris.fileformats.netcdf._action_activation_stats` to list the
-        facts.
-
-        """
-        return self.facts
-
     def fact_list(self, fact_name):
         """
         Return the facts (arg-lists) for one fact name.
 
         A shorthand form used only by the new 'actions' routines.
 
         AKA 'case-specific-facts', in the original.
-        Roughly "return self.facts.entity_lists[fact_name]".
+        Roughly = "self.facts.entity_lists[fact_name].case_specific_facts".
 
         """
         return self.facts.sect_facts(fact_name)