More explicit TODOs.
authorRobert Bradshaw <robertwb@math.washington.edu>
Thu, 13 Aug 2009 08:12:44 +0000 (01:12 -0700)
committerRobert Bradshaw <robertwb@math.washington.edu>
Thu, 13 Aug 2009 08:12:44 +0000 (01:12 -0700)
Cython/Compiler/ExprNodes.py
Cython/Compiler/PyrexTypes.py

index 2098e05ee152f150f8f2229f83b59afe24e8f710..d28cf9222de99556db2f5f9af60253f51633bb53 100755 (executable)
@@ -3928,7 +3928,7 @@ class UnopNode(ExprNode):
         "++":       u"__inc__",
         "--":       u"__dec__",
         "*":        u"__deref__",
-        "!":        u"__not__"
+        "!":        u"__not__" # TODO(danilo): Also handle in NotNode.
     }
         
 
index ba521a307f39b570516841f7dd781efd769007a5..b4b64726b4897786445f4698fdb8f975776af52a 100755 (executable)
@@ -109,6 +109,7 @@ class PyrexType(BaseType):
         return self
     
     def specialize(self, values):
+        # TODO(danilo): Override wherever it makes sense.
         return self
     
     def literal_code(self, value):
@@ -1396,7 +1397,6 @@ class CppClassType(CType):
         self.template_type = template_type
 
     def specialize_here(self, pos, template_values = None):
-        # TODO: cache for efficiency
         if self.templates is None:
             error(pos, "'%s' type is not a template" % self);
             return PyrexTypes.error_type
@@ -1407,7 +1407,7 @@ class CppClassType(CType):
         return self.specialize(dict(zip(self.templates, template_values)))
     
     def specialize(self, values):
-        # TODO: cache for efficiency
+        # TODO(danilo): Cache for efficiency.
         template_values = [t.specialize(values) for t in self.templates]
         return CppClassType(self.name, self.scope.specialize(values), self.cname, self.base_classes, 
                             template_values, template_type=self)
@@ -1425,7 +1425,7 @@ class CppClassType(CType):
         return "%s %s%s" % (name, entity_code, templates)
 
     def is_subclass(self, other_type):
-        # TODO: handle templates
+        # TODO(danilo): Handle templates.
         if self.same_as_resolved_type(other_type):
             return 1
         for base_class in self.base_classes: