removed dead comments

Author: Joshua Reich

pyretic/core/classifier.py

@@ -145,8 +145,6 @@ def __add__(c1, c2):
         def _cross(r1,r2):
             intersection = r1.match.intersect(r2.match)
             if intersection != drop:
-                # TODO (josh) logic for detecting when sets of actions can't be combined
-                # e.g., [modify(dstip='10.0.0.1'),fwd(1)] + [modify(srcip='10.0.0.2'),fwd(2)]
                 actions = r1.actions | r2.actions
                 return Rule(intersection, actions)
             else:
@@ -200,8 +198,6 @@ def _commute_test(act, pkts):
                     return identity
                 return match(**new_match_dict)
             else:
-                # TODO (cole) use compile error.
-                # TODO (cole) what actions are allowable?
                 raise TypeError
 
         # sequentially compose actions.  a1 must be a

pyretic/core/runtime.py

@@ -330,6 +330,9 @@ def install_classifier(self, classifier):
 
         ### CLASSIFIER TRANSFORMS 
 
+        # TODO (josh) logic for detecting action sets that can't be compiled
+        # e.g., {modify(dstip='10.0.0.1',outport=1),modify(srcip='10.0.0.2',outport=2)]
+
         def remove_identity(classifier):
             """
             Removes identity policies from the action list.
@@ -339,8 +342,6 @@ def remove_identity(classifier):
             :returns: the output classifier
             :rtype: Classifier
             """
-            # DISCUSS (cole): convert identity to inport rather
-            # than drop?
             return Classifier(Rule(rule.match,
                                    filter(lambda a: a != identity,rule.actions))
                               for rule in classifier.rules)