[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