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

[Stephan Richter]

Log message for revision 26469:
  Converted XXX to TODO.
  


Changed:
  U   Zope3/trunk/src/zope/publisher/base.py
  U   Zope3/trunk/src/zope/publisher/browser.py
  U   Zope3/trunk/src/zope/publisher/http.py
  U   Zope3/trunk/src/zope/publisher/interfaces/http.py
  U   Zope3/trunk/src/zope/publisher/tests/test_browserresponse.py


-=-
Modified: Zope3/trunk/src/zope/publisher/base.py
===================================================================
--- Zope3/trunk/src/zope/publisher/base.py	2004-07-13 16:51:32 UTC (rev 26468)
+++ Zope3/trunk/src/zope/publisher/base.py	2004-07-13 16:52:22 UTC (rev 26469)
@@ -317,7 +317,7 @@
         if body is None:
             s = self._body_instream
             if s is None:
-                return None # XXX what should be returned here?
+                return None # TODO: what should be returned here?
             p = s.tell()
             s.seek(0)
             body = s.read()

Modified: Zope3/trunk/src/zope/publisher/browser.py
===================================================================
--- Zope3/trunk/src/zope/publisher/browser.py	2004-07-13 16:51:32 UTC (rev 26468)
+++ Zope3/trunk/src/zope/publisher/browser.py	2004-07-13 16:52:22 UTC (rev 26469)
@@ -654,7 +654,7 @@
                 c += ';charset=' + self._charset
             self.setHeader('content-type', c)
             self.setHeader('x-content-type-warning', 'guessed from content')
-            # XXX emit a warning once all page templates are changed to
+            # TODO: emit a warning once all page templates are changed to
             # specify their content type explicitly.
 
         body = self.__insertBase(body)
@@ -721,10 +721,10 @@
                 base += '/'
             location = base + location
 
-        # XXX: HTTP redirects must provide an absolute location, see
-        #      http://www.w3.org/Protocols/rfc2616/rfc2616-sec14.html#sec14.30
-        #      So, what if location is relative and base is unknown?  Uncomment
-        #      the following and you'll see that it actually happens.
+        # TODO: HTTP redirects must provide an absolute location, see
+        #       http://www.w3.org/Protocols/rfc2616/rfc2616-sec14.html#sec14.30
+        #       So, what if location is relative and base is unknown?  Uncomment
+        #       the following and you'll see that it actually happens.
         #
         # if isRelative(str(location)):
         #     raise AssertionError('Cannot determine absolute location')

Modified: Zope3/trunk/src/zope/publisher/http.py
===================================================================
--- Zope3/trunk/src/zope/publisher/http.py	2004-07-13 16:51:32 UTC (rev 26468)
+++ Zope3/trunk/src/zope/publisher/http.py	2004-07-13 16:52:22 UTC (rev 26469)
@@ -853,7 +853,8 @@
                     cookie = '%s; Secure' % cookie
             cookie_list.append(cookie)
 
-        # XXX: Should really check size of cookies here!
+        # TODO: Should really check size of cookies here!
+        # Why?
 
         return cookie_list
 

Modified: Zope3/trunk/src/zope/publisher/interfaces/http.py
===================================================================
--- Zope3/trunk/src/zope/publisher/interfaces/http.py	2004-07-13 16:51:32 UTC (rev 26468)
+++ Zope3/trunk/src/zope/publisher/interfaces/http.py	2004-07-13 16:52:22 UTC (rev 26469)
@@ -208,7 +208,7 @@
 
 class IHTTPCredentials(Interface):
 
-    # XXX Eventially this will be a different method
+    # TODO: Eventially this will be a different method
     def _authUserPW():
         """Return (login, password) if there are basic credentials;
         return None if there aren't."""

Modified: Zope3/trunk/src/zope/publisher/tests/test_browserresponse.py
===================================================================
--- Zope3/trunk/src/zope/publisher/tests/test_browserresponse.py	2004-07-13 16:51:32 UTC (rev 26468)
+++ Zope3/trunk/src/zope/publisher/tests/test_browserresponse.py	2004-07-13 16:52:22 UTC (rev 26469)
@@ -13,8 +13,6 @@
 ##############################################################################
 """Browser response tests
 
-XXX longer description goes here.
-
 $Id$
 """
 
@@ -23,7 +21,7 @@
 from StringIO import StringIO
 from zope.interface.verify import verifyObject
 
-# XXX Waaa need more tests
+# TODO: Waaa need more tests
 
 class TestBrowserResponse(TestCase):