[Zope3-checkins] SVN: Zope3/trunk/src/zope/tal/ Converted XXX to TODO.

Stephan Richter srichter at cosmos.phy.tufts.edu
Tue Jul 13 12:56:45 EDT 2004


Log message for revision 26473:
  Converted XXX to TODO.
  


Changed:
  U   Zope3/trunk/src/zope/tal/dummyengine.py
  U   Zope3/trunk/src/zope/tal/setpath.py
  U   Zope3/trunk/src/zope/tal/talgenerator.py
  U   Zope3/trunk/src/zope/tal/talinterpreter.py
  U   Zope3/trunk/src/zope/tal/tests/markbench.py
  U   Zope3/trunk/src/zope/tal/tests/test_talinterpreter.py


-=-
Modified: Zope3/trunk/src/zope/tal/dummyengine.py
===================================================================
--- Zope3/trunk/src/zope/tal/dummyengine.py	2004-07-13 16:56:24 UTC (rev 26472)
+++ Zope3/trunk/src/zope/tal/dummyengine.py	2004-07-13 16:56:45 UTC (rev 26473)
@@ -144,11 +144,11 @@
         return text
 
     def evaluateStructure(self, expr):
-        # XXX Should return None or a DOM tree
+        # TODO Should return None or a DOM tree
         return self.evaluate(expr)
 
     def evaluateSequence(self, expr):
-        # XXX Should return a sequence
+        # TODO: Should return a sequence
         return self.evaluate(expr)
 
     def evaluateMacro(self, macroName):

Modified: Zope3/trunk/src/zope/tal/setpath.py
===================================================================
--- Zope3/trunk/src/zope/tal/setpath.py	2004-07-13 16:56:24 UTC (rev 26472)
+++ Zope3/trunk/src/zope/tal/setpath.py	2004-07-13 16:56:45 UTC (rev 26473)
@@ -11,15 +11,14 @@
 # FOR A PARTICULAR PURPOSE.
 #
 ##############################################################################
-"""
-Read a module search path from .path file.
+"""Read a module search path from .path file.
 
 If .path file isn't found in the directory of the setpath.py module, then try
 to import ZODB.  If that succeeds, we assume the path is already set up
 correctly.  If that import fails, an IOError is raised.
 """
 
-# XXX Why does this want to find ZODB ???
+# TODO: Why does this want to find ZODB ???
 
 import os
 import sys

Modified: Zope3/trunk/src/zope/tal/talgenerator.py
===================================================================
--- Zope3/trunk/src/zope/tal/talgenerator.py	2004-07-13 16:56:24 UTC (rev 26472)
+++ Zope3/trunk/src/zope/tal/talgenerator.py	2004-07-13 16:56:45 UTC (rev 26473)
@@ -556,7 +556,7 @@
             # Hack to include preceding whitespace in the loop program
             repeatWhitespace = self.unEmitNewlineWhitespace()
         if position != (None, None):
-            # XXX at some point we should insist on a non-trivial position
+            # TODO: at some point we should insist on a non-trivial position
             self.emit("setPosition", position)
         if self.inMacroUse:
             if fillSlot:

Modified: Zope3/trunk/src/zope/tal/talinterpreter.py
===================================================================
--- Zope3/trunk/src/zope/tal/talinterpreter.py	2004-07-13 16:56:24 UTC (rev 26472)
+++ Zope3/trunk/src/zope/tal/talinterpreter.py	2004-07-13 16:56:45 UTC (rev 26473)
@@ -31,7 +31,7 @@
     # List of Boolean attributes in HTML that should be rendered in
     # minimized form (e.g. <img ismap> rather than <img ismap="">)
     # From http://www.w3.org/TR/xhtml1/#guidelines (C.10)
-    # XXX The problem with this is that this is not valid XML and
+    # TODO: The problem with this is that this is not valid XML and
     # can't be parsed back!
     "compact", "nowrap", "ismap", "declare", "noshade", "checked",
     "disabled", "readonly", "multiple", "selected", "noresize",
@@ -105,7 +105,7 @@
         2. Comments are not allowed in XML documents before the <?xml?>
            declaration.
 
-    For performance reasons (XXX premature optimization?) instead of checking
+    For performance reasons (TODO: premature optimization?) instead of checking
     the value of _pending_source_annotation on every write to the output
     stream, the _stream_write attribute is changed to point to
     _annotated_stream_write method whenever _pending_source_annotation is
@@ -142,7 +142,7 @@
             -1 (which is the default setting) enables showtal when TAL
             processing is disabled, and disables showtal when TAL processing is
             enabled.  Note that you must use 0, 1, or -1; true boolean values
-            are not supported (XXX why?).
+            are not supported (TODO: why?).
 
             strictinsert -- enable TAL processing and stricter HTML/XML
             checking on text produced by structure inserts (on by default).
@@ -643,8 +643,8 @@
         if len(stuff) > 2:
             obj = self.engine.evaluate(stuff[2])
         xlated_msgid = self.translate(msgid, default, i18ndict, obj)
-        # XXX I can't decide whether we want to cgi escape the translated
-        # string or not.  OT1H not doing this could introduce a cross-site
+        # TODO: I can't decide whether we want to cgi escape the translated
+        # string or not.  OTOH not doing this could introduce a cross-site
         # scripting vector by allowing translators to sneak JavaScript into
         # translations.  OTOH, for implicit interpolation values, we don't
         # want to escape stuff like ${name} <= "<b>Timmy</b>".
@@ -723,7 +723,7 @@
             i18ndict.update(obj)
         if not self.i18nInterpolate:
             return msgid
-        # XXX We need to pass in one of context or target_language
+        # TODO: We need to pass in one of context or target_language
         return self.engine.translate(msgid, self.i18nContext.domain,
                                      i18ndict, default=default)
 

Modified: Zope3/trunk/src/zope/tal/tests/markbench.py
===================================================================
--- Zope3/trunk/src/zope/tal/tests/markbench.py	2004-07-13 16:56:24 UTC (rev 26472)
+++ Zope3/trunk/src/zope/tal/tests/markbench.py	2004-07-13 16:56:45 UTC (rev 26473)
@@ -154,7 +154,7 @@
         rc = os.spawnl(os.P_WAIT, sys.executable,
                        sys.executable, "test.py", "zope.tal.tests")
         if rc > 0:
-            # XXX Failing tests don't cause test.py to report an
+            # TODO: Failing tests don't cause test.py to report an
             # error; not sure why.  ;-(
             sys.exit(rc)
         elif rc < 0:

Modified: Zope3/trunk/src/zope/tal/tests/test_talinterpreter.py
===================================================================
--- Zope3/trunk/src/zope/tal/tests/test_talinterpreter.py	2004-07-13 16:56:24 UTC (rev 26472)
+++ Zope3/trunk/src/zope/tal/tests/test_talinterpreter.py	2004-07-13 16:56:45 UTC (rev 26473)
@@ -437,7 +437,7 @@
     suite.addTest(unittest.makeSuite(I18NCornerTestCase))
     suite.addTest(unittest.makeSuite(TestSourceAnnotations))
 
-    # XXX: Deactivated test, since we have not found a solution for this and
+    # TODO: Deactivated test, since we have not found a solution for this and
     # it is a deep and undocumented HTML parser issue.
     # Fred is looking into this.
     #suite.addTest(unittest.makeSuite(MacroFunkyErrorTest))



More information about the Zope3-Checkins mailing list