fix comments; split a function so we can have separate parse error messages for each at some point

annevk · annevk · commit 8e72ade6003d · 2006-12-29T17:38:13.000Z
--HG--
extra : convert_revision : svn%3Aacbfec75-9323-0410-a652-858a13e371e0/trunk%40373
diff --git a/src/parser.py b/src/parser.py
@@ -53,9 +53,10 @@ def parse(self, stream, innerHTML=False):
         """Stream should be a stream of unicode bytes. Character encoding
         issues have not yet been dealt with."""
 
-        #XXX - need to ensure the tree is reset here
+        # XXX - need to ensure the tree is reset here
+        # Why? -- anne
 
-        # We don't actually support inner HTML yet but this should allow
+        # We don't actually support innerHTML yet but this should allow
         # assertations
         self.innerHTML = innerHTML
 
@@ -1559,24 +1560,27 @@ def __init__(self, parser, tree):
             ("html", self.startTagHtml),
             ("noframes", self.startTagNoframes)
         ])
-        self.startTagHandler.default = self.tagOther
+        self.startTagHandler.default = self.startTagOther
 
         self.endTagHandler = utils.MethodDispatcher([
             ("html", self.endTagHtml)
         ])
-        self.endTagHandler.default = self.tagOther
+        self.endTagHandler.default = self.endTagOther
 
     def processCharacters(self, data):
         self.parser.parseError()
 
     def startTagNoframes(self, name, attributes):
         self.parser.phases["inBody"].processStartTag(name, attributes)
 
+    def startTagOther(self, name, attributes):
+        self.parser.parseError()
+
     def endTagHtml(self, name):
         self.parser.lastPhase = self.parser.phase
         self.parser.phase = self.parser.phases["trailingEnd"]
 
-    def tagOther(self, name, attributes={}):
+    def endTagOther(self, name):
         self.parser.parseError()
 
 
