From 1a752708f941ddf542dddebe7c7c14c3c0ff5c2f Mon Sep 17 00:00:00 2001
From: rickg <rickg@18797224-902f-48f8-a5cc-f745e15eee43>
Date: Sat, 20 Jun 1998 00:12:10 +0000
Subject: [PATCH] added support for empty (XML) tags

git-svn-id: svn://10.0.0.236/trunk@4178 18797224-902f-48f8-a5cc-f745e15eee43
---
 mozilla/htmlparser/src/CNavDTD.cpp            | 23 ++++++++++++++----
 mozilla/htmlparser/src/CNavDTD.h              |  2 +-
 mozilla/htmlparser/src/COtherDTD.cpp          |  6 ++---
 mozilla/htmlparser/src/COtherDTD.h            |  2 +-
 mozilla/htmlparser/src/nsHTMLTokens.cpp       | 24 +++++++++++++++++++
 mozilla/htmlparser/src/nsHTMLTokens.h         |  6 +++--
 mozilla/htmlparser/src/nsParser.cpp           |  9 +------
 mozilla/htmlparser/src/nsScanner.cpp          |  2 +-
 mozilla/parser/htmlparser/src/CNavDTD.cpp     | 23 ++++++++++++++----
 mozilla/parser/htmlparser/src/CNavDTD.h       |  2 +-
 mozilla/parser/htmlparser/src/COtherDTD.cpp   |  6 ++---
 mozilla/parser/htmlparser/src/COtherDTD.h     |  2 +-
 .../parser/htmlparser/src/nsHTMLTokens.cpp    | 24 +++++++++++++++++++
 mozilla/parser/htmlparser/src/nsHTMLTokens.h  |  6 +++--
 mozilla/parser/htmlparser/src/nsParser.cpp    |  9 +------
 mozilla/parser/htmlparser/src/nsScanner.cpp   |  2 +-
 16 files changed, 106 insertions(+), 42 deletions(-)

diff --git a/mozilla/htmlparser/src/CNavDTD.cpp b/mozilla/htmlparser/src/CNavDTD.cpp
index 8d17415a202..08650f5ddef 100644
--- a/mozilla/htmlparser/src/CNavDTD.cpp
+++ b/mozilla/htmlparser/src/CNavDTD.cpp
@@ -2393,17 +2393,30 @@ PRInt32 CNavDTD::ConsumeTag(PRUnichar aChar,CScanner& aScanner,CToken*& aToken)
  *  @param   aScanner: see nsScanner.h
  *  @return  
  */
-PRInt32 CNavDTD::ConsumeAttributes(PRUnichar aChar,CScanner& aScanner,CToken*& aToken) {
+PRInt32 CNavDTD::ConsumeAttributes(PRUnichar aChar,CScanner& aScanner,CStartToken* aToken) {
   PRBool done=PR_FALSE;
   PRInt32 result=kNoError;
   nsAutoString as("");
   PRInt16 theAttrCount=0;
 
   while((!done) && (result==kNoError)) {
-    CToken* theToken= new CAttributeToken(as);
+    CAttributeToken* theToken= new CAttributeToken(as);
     if(theToken){
       result=theToken->Consume(aChar,aScanner);  //tell new token to finish consuming text...    
-      if(kNoError==result){
+
+      //Much as I hate to do this, here's some special case code.
+      //This handles the case of empty-tags in XML. Our last
+      //attribute token will come through with a text value of ""
+      //and a textkey of "/". We should destroy it, and tell the 
+      //start token it was empty.
+      nsString& key=theToken->GetKey();
+      nsString& text=theToken->GetText();
+      if((key[0]==kForwardSlash) && (0==text.Length())){
+        //tada! our special case! Treat it like an empty start tag...
+        aToken->SetEmpty(PR_TRUE);
+        delete theToken;
+      }
+      else if(kNoError==result){
         theAttrCount++;
         mTokenDeque.Push(theToken);
       }//if
@@ -2459,12 +2472,12 @@ PRInt32 CNavDTD::ConsumeStartTag(PRUnichar aChar,CScanner& aScanner,CToken*& aTo
   PRInt32 result=kNoError;
 
   aToken=new CStartToken(nsAutoString(""));
-
+  
   if(aToken) {
     result= aToken->Consume(aChar,aScanner);  //tell new token to finish consuming text...    
     if(kNoError==result) {
       if(((CStartToken*)aToken)->IsAttributed()) {
-        result=ConsumeAttributes(aChar,aScanner,aToken);
+        result=ConsumeAttributes(aChar,aScanner,(CStartToken*)aToken);
       }
       //now that that's over with, we have one more problem to solve.
       //In the case that we just read a <SCRIPT> or <STYLE> tags, we should go and
diff --git a/mozilla/htmlparser/src/CNavDTD.h b/mozilla/htmlparser/src/CNavDTD.h
index 410485673c4..e8b33f47e67 100644
--- a/mozilla/htmlparser/src/CNavDTD.h
+++ b/mozilla/htmlparser/src/CNavDTD.h
@@ -613,7 +613,7 @@ private:
      * @param   aToken is the next token (or null)
      * @return  error code
      */
-    PRInt32     ConsumeAttributes(PRUnichar aChar,CScanner& aScanner,CToken*& aToken);
+    PRInt32     ConsumeAttributes(PRUnichar aChar,CScanner& aScanner,CStartToken* aToken);
     
     /**
      * Retrieve a sequence of text from given scanner.
diff --git a/mozilla/htmlparser/src/COtherDTD.cpp b/mozilla/htmlparser/src/COtherDTD.cpp
index e0fe0db6ad6..2b8f0e6276c 100644
--- a/mozilla/htmlparser/src/COtherDTD.cpp
+++ b/mozilla/htmlparser/src/COtherDTD.cpp
@@ -2393,7 +2393,7 @@ PRInt32 COtherDTD::ConsumeTag(PRUnichar aChar,CScanner& aScanner,CToken*& aToken
  *  @param   aScanner: see nsScanner.h
  *  @return  
  */
-PRInt32 COtherDTD::ConsumeAttributes(PRUnichar aChar,CScanner& aScanner,CToken*& aToken) {
+PRInt32 COtherDTD::ConsumeAttributes(PRUnichar aChar,CScanner& aScanner,CStartToken* aToken) {
   PRBool done=PR_FALSE;
   PRInt32 result=kNoError;
   nsAutoString as("");
@@ -2459,12 +2459,12 @@ PRInt32 COtherDTD::ConsumeStartTag(PRUnichar aChar,CScanner& aScanner,CToken*& a
   PRInt32 result=kNoError;
 
   aToken=new CStartToken(nsAutoString(""));
-
+  
   if(aToken) {
     result= aToken->Consume(aChar,aScanner);  //tell new token to finish consuming text...    
     if(kNoError==result) {
       if(((CStartToken*)aToken)->IsAttributed()) {
-        result=ConsumeAttributes(aChar,aScanner,aToken);
+        result=ConsumeAttributes(aChar,aScanner,(CStartToken*)aToken);
       }
       //now that that's over with, we have one more problem to solve.
       //In the case that we just read a <SCRIPT> or <STYLE> tags, we should go and
diff --git a/mozilla/htmlparser/src/COtherDTD.h b/mozilla/htmlparser/src/COtherDTD.h
index 1cd27f7cd46..026e5ea18d5 100644
--- a/mozilla/htmlparser/src/COtherDTD.h
+++ b/mozilla/htmlparser/src/COtherDTD.h
@@ -614,7 +614,7 @@ private:
      * @param   aToken is the next token (or null)
      * @return  error code
      */
-    PRInt32     ConsumeAttributes(PRUnichar aChar,CScanner& aScanner,CToken*& aToken);
+    PRInt32     ConsumeAttributes(PRUnichar aChar,CScanner& aScanner,CStartToken* aToken);
     
     /**
      * Retrieve a sequence of text from given scanner.
diff --git a/mozilla/htmlparser/src/nsHTMLTokens.cpp b/mozilla/htmlparser/src/nsHTMLTokens.cpp
index 288c798e8bd..250d760b033 100644
--- a/mozilla/htmlparser/src/nsHTMLTokens.cpp
+++ b/mozilla/htmlparser/src/nsHTMLTokens.cpp
@@ -285,6 +285,7 @@ CHTMLToken::CHTMLToken(eHTMLTags aTag) : CToken(GetTagName(aTag)) {
  */
 CStartToken::CStartToken(const nsString& aName) : CHTMLToken(aName) {
   mAttributed=PR_FALSE;
+  mEmpty=PR_FALSE;
 }
 
 /*
@@ -296,6 +297,7 @@ CStartToken::CStartToken(const nsString& aName) : CHTMLToken(aName) {
  */
 CStartToken::CStartToken(eHTMLTags aTag) : CHTMLToken(aTag) {
   mAttributed=PR_FALSE;
+  mEmpty=PR_FALSE;
 }
 
 /*
@@ -355,6 +357,28 @@ PRBool CStartToken::IsAttributed(void) {
   return mAttributed;
 }
 
+/*
+ *  
+ *  
+ *  @update  gess 3/25/98
+ *  @param   
+ *  @return  
+ */
+void CStartToken::SetEmpty(PRBool aValue) {
+  mEmpty=aValue;
+}
+
+/*
+ *  
+ *  
+ *  @update  gess 3/25/98
+ *  @param   
+ *  @return  
+ */
+PRBool CStartToken::IsEmpty(void) {
+  return mEmpty;
+}
+
 /*
  *  Consume the identifier portion of the start tag
  *  
diff --git a/mozilla/htmlparser/src/nsHTMLTokens.h b/mozilla/htmlparser/src/nsHTMLTokens.h
index dadac171edc..a96c1ee3579 100644
--- a/mozilla/htmlparser/src/nsHTMLTokens.h
+++ b/mozilla/htmlparser/src/nsHTMLTokens.h
@@ -155,13 +155,15 @@ class CStartToken: public CHTMLToken {
     virtual PRInt32     GetTypeID(void);
     virtual const char* GetClassName(void);
     virtual PRInt32     GetTokenType(void);
-            void        SetAttributed(PRBool aValue);
             PRBool      IsAttributed(void);
+            void        SetAttributed(PRBool aValue);
+            PRBool      IsEmpty(void);
+            void        SetEmpty(PRBool aValue);
     virtual void        DebugDumpSource(ostream& out);
   
   protected:
             PRBool      mAttributed;      
-
+            PRBool      mEmpty;      
 };
 
 
diff --git a/mozilla/htmlparser/src/nsParser.cpp b/mozilla/htmlparser/src/nsParser.cpp
index 2434e364dbe..ad38e10e9d6 100644
--- a/mozilla/htmlparser/src/nsParser.cpp
+++ b/mozilla/htmlparser/src/nsParser.cpp
@@ -16,7 +16,7 @@
  * Reserved.
  */
   
-//#define __INCREMENTAL 1
+
 
 #include "nsParser.h"
 #include "nsIContentSink.h" 
@@ -31,13 +31,6 @@
 #include "nsIInputStream.h"
 #include "nsIParserFilter.h"
 
-/* UNCOMMENT THIS IF STUFF STOPS WORKING...
-#ifdef XP_PC
-#include <direct.h> //this is here for debug reasons...
-#endif
-#include <time.h>
-#include "prmem.h"
-*/
 
 static NS_DEFINE_IID(kISupportsIID, NS_ISUPPORTS_IID);                 
 static NS_DEFINE_IID(kClassIID, NS_PARSER_IID); 
diff --git a/mozilla/htmlparser/src/nsScanner.cpp b/mozilla/htmlparser/src/nsScanner.cpp
index ca6feea325f..72337972685 100644
--- a/mozilla/htmlparser/src/nsScanner.cpp
+++ b/mozilla/htmlparser/src/nsScanner.cpp
@@ -322,7 +322,7 @@ PRInt32 CScanner::GetChar(PRUnichar& aChar) {
  *  @param   
  *  @return  
  */
-PRInt32 CScanner::Peek(PRUnichar& aChar){
+PRInt32 CScanner::Peek(PRUnichar& aChar) {
   PRInt32 result=Eof();
   if(!result) {
     aChar=mBuffer[mOffset];        
diff --git a/mozilla/parser/htmlparser/src/CNavDTD.cpp b/mozilla/parser/htmlparser/src/CNavDTD.cpp
index 8d17415a202..08650f5ddef 100644
--- a/mozilla/parser/htmlparser/src/CNavDTD.cpp
+++ b/mozilla/parser/htmlparser/src/CNavDTD.cpp
@@ -2393,17 +2393,30 @@ PRInt32 CNavDTD::ConsumeTag(PRUnichar aChar,CScanner& aScanner,CToken*& aToken)
  *  @param   aScanner: see nsScanner.h
  *  @return  
  */
-PRInt32 CNavDTD::ConsumeAttributes(PRUnichar aChar,CScanner& aScanner,CToken*& aToken) {
+PRInt32 CNavDTD::ConsumeAttributes(PRUnichar aChar,CScanner& aScanner,CStartToken* aToken) {
   PRBool done=PR_FALSE;
   PRInt32 result=kNoError;
   nsAutoString as("");
   PRInt16 theAttrCount=0;
 
   while((!done) && (result==kNoError)) {
-    CToken* theToken= new CAttributeToken(as);
+    CAttributeToken* theToken= new CAttributeToken(as);
     if(theToken){
       result=theToken->Consume(aChar,aScanner);  //tell new token to finish consuming text...    
-      if(kNoError==result){
+
+      //Much as I hate to do this, here's some special case code.
+      //This handles the case of empty-tags in XML. Our last
+      //attribute token will come through with a text value of ""
+      //and a textkey of "/". We should destroy it, and tell the 
+      //start token it was empty.
+      nsString& key=theToken->GetKey();
+      nsString& text=theToken->GetText();
+      if((key[0]==kForwardSlash) && (0==text.Length())){
+        //tada! our special case! Treat it like an empty start tag...
+        aToken->SetEmpty(PR_TRUE);
+        delete theToken;
+      }
+      else if(kNoError==result){
         theAttrCount++;
         mTokenDeque.Push(theToken);
       }//if
@@ -2459,12 +2472,12 @@ PRInt32 CNavDTD::ConsumeStartTag(PRUnichar aChar,CScanner& aScanner,CToken*& aTo
   PRInt32 result=kNoError;
 
   aToken=new CStartToken(nsAutoString(""));
-
+  
   if(aToken) {
     result= aToken->Consume(aChar,aScanner);  //tell new token to finish consuming text...    
     if(kNoError==result) {
       if(((CStartToken*)aToken)->IsAttributed()) {
-        result=ConsumeAttributes(aChar,aScanner,aToken);
+        result=ConsumeAttributes(aChar,aScanner,(CStartToken*)aToken);
       }
       //now that that's over with, we have one more problem to solve.
       //In the case that we just read a <SCRIPT> or <STYLE> tags, we should go and
diff --git a/mozilla/parser/htmlparser/src/CNavDTD.h b/mozilla/parser/htmlparser/src/CNavDTD.h
index 410485673c4..e8b33f47e67 100644
--- a/mozilla/parser/htmlparser/src/CNavDTD.h
+++ b/mozilla/parser/htmlparser/src/CNavDTD.h
@@ -613,7 +613,7 @@ private:
      * @param   aToken is the next token (or null)
      * @return  error code
      */
-    PRInt32     ConsumeAttributes(PRUnichar aChar,CScanner& aScanner,CToken*& aToken);
+    PRInt32     ConsumeAttributes(PRUnichar aChar,CScanner& aScanner,CStartToken* aToken);
     
     /**
      * Retrieve a sequence of text from given scanner.
diff --git a/mozilla/parser/htmlparser/src/COtherDTD.cpp b/mozilla/parser/htmlparser/src/COtherDTD.cpp
index e0fe0db6ad6..2b8f0e6276c 100644
--- a/mozilla/parser/htmlparser/src/COtherDTD.cpp
+++ b/mozilla/parser/htmlparser/src/COtherDTD.cpp
@@ -2393,7 +2393,7 @@ PRInt32 COtherDTD::ConsumeTag(PRUnichar aChar,CScanner& aScanner,CToken*& aToken
  *  @param   aScanner: see nsScanner.h
  *  @return  
  */
-PRInt32 COtherDTD::ConsumeAttributes(PRUnichar aChar,CScanner& aScanner,CToken*& aToken) {
+PRInt32 COtherDTD::ConsumeAttributes(PRUnichar aChar,CScanner& aScanner,CStartToken* aToken) {
   PRBool done=PR_FALSE;
   PRInt32 result=kNoError;
   nsAutoString as("");
@@ -2459,12 +2459,12 @@ PRInt32 COtherDTD::ConsumeStartTag(PRUnichar aChar,CScanner& aScanner,CToken*& a
   PRInt32 result=kNoError;
 
   aToken=new CStartToken(nsAutoString(""));
-
+  
   if(aToken) {
     result= aToken->Consume(aChar,aScanner);  //tell new token to finish consuming text...    
     if(kNoError==result) {
       if(((CStartToken*)aToken)->IsAttributed()) {
-        result=ConsumeAttributes(aChar,aScanner,aToken);
+        result=ConsumeAttributes(aChar,aScanner,(CStartToken*)aToken);
       }
       //now that that's over with, we have one more problem to solve.
       //In the case that we just read a <SCRIPT> or <STYLE> tags, we should go and
diff --git a/mozilla/parser/htmlparser/src/COtherDTD.h b/mozilla/parser/htmlparser/src/COtherDTD.h
index 1cd27f7cd46..026e5ea18d5 100644
--- a/mozilla/parser/htmlparser/src/COtherDTD.h
+++ b/mozilla/parser/htmlparser/src/COtherDTD.h
@@ -614,7 +614,7 @@ private:
      * @param   aToken is the next token (or null)
      * @return  error code
      */
-    PRInt32     ConsumeAttributes(PRUnichar aChar,CScanner& aScanner,CToken*& aToken);
+    PRInt32     ConsumeAttributes(PRUnichar aChar,CScanner& aScanner,CStartToken* aToken);
     
     /**
      * Retrieve a sequence of text from given scanner.
diff --git a/mozilla/parser/htmlparser/src/nsHTMLTokens.cpp b/mozilla/parser/htmlparser/src/nsHTMLTokens.cpp
index 288c798e8bd..250d760b033 100644
--- a/mozilla/parser/htmlparser/src/nsHTMLTokens.cpp
+++ b/mozilla/parser/htmlparser/src/nsHTMLTokens.cpp
@@ -285,6 +285,7 @@ CHTMLToken::CHTMLToken(eHTMLTags aTag) : CToken(GetTagName(aTag)) {
  */
 CStartToken::CStartToken(const nsString& aName) : CHTMLToken(aName) {
   mAttributed=PR_FALSE;
+  mEmpty=PR_FALSE;
 }
 
 /*
@@ -296,6 +297,7 @@ CStartToken::CStartToken(const nsString& aName) : CHTMLToken(aName) {
  */
 CStartToken::CStartToken(eHTMLTags aTag) : CHTMLToken(aTag) {
   mAttributed=PR_FALSE;
+  mEmpty=PR_FALSE;
 }
 
 /*
@@ -355,6 +357,28 @@ PRBool CStartToken::IsAttributed(void) {
   return mAttributed;
 }
 
+/*
+ *  
+ *  
+ *  @update  gess 3/25/98
+ *  @param   
+ *  @return  
+ */
+void CStartToken::SetEmpty(PRBool aValue) {
+  mEmpty=aValue;
+}
+
+/*
+ *  
+ *  
+ *  @update  gess 3/25/98
+ *  @param   
+ *  @return  
+ */
+PRBool CStartToken::IsEmpty(void) {
+  return mEmpty;
+}
+
 /*
  *  Consume the identifier portion of the start tag
  *  
diff --git a/mozilla/parser/htmlparser/src/nsHTMLTokens.h b/mozilla/parser/htmlparser/src/nsHTMLTokens.h
index dadac171edc..a96c1ee3579 100644
--- a/mozilla/parser/htmlparser/src/nsHTMLTokens.h
+++ b/mozilla/parser/htmlparser/src/nsHTMLTokens.h
@@ -155,13 +155,15 @@ class CStartToken: public CHTMLToken {
     virtual PRInt32     GetTypeID(void);
     virtual const char* GetClassName(void);
     virtual PRInt32     GetTokenType(void);
-            void        SetAttributed(PRBool aValue);
             PRBool      IsAttributed(void);
+            void        SetAttributed(PRBool aValue);
+            PRBool      IsEmpty(void);
+            void        SetEmpty(PRBool aValue);
     virtual void        DebugDumpSource(ostream& out);
   
   protected:
             PRBool      mAttributed;      
-
+            PRBool      mEmpty;      
 };
 
 
diff --git a/mozilla/parser/htmlparser/src/nsParser.cpp b/mozilla/parser/htmlparser/src/nsParser.cpp
index 2434e364dbe..ad38e10e9d6 100644
--- a/mozilla/parser/htmlparser/src/nsParser.cpp
+++ b/mozilla/parser/htmlparser/src/nsParser.cpp
@@ -16,7 +16,7 @@
  * Reserved.
  */
   
-//#define __INCREMENTAL 1
+
 
 #include "nsParser.h"
 #include "nsIContentSink.h" 
@@ -31,13 +31,6 @@
 #include "nsIInputStream.h"
 #include "nsIParserFilter.h"
 
-/* UNCOMMENT THIS IF STUFF STOPS WORKING...
-#ifdef XP_PC
-#include <direct.h> //this is here for debug reasons...
-#endif
-#include <time.h>
-#include "prmem.h"
-*/
 
 static NS_DEFINE_IID(kISupportsIID, NS_ISUPPORTS_IID);                 
 static NS_DEFINE_IID(kClassIID, NS_PARSER_IID); 
diff --git a/mozilla/parser/htmlparser/src/nsScanner.cpp b/mozilla/parser/htmlparser/src/nsScanner.cpp
index ca6feea325f..72337972685 100644
--- a/mozilla/parser/htmlparser/src/nsScanner.cpp
+++ b/mozilla/parser/htmlparser/src/nsScanner.cpp
@@ -322,7 +322,7 @@ PRInt32 CScanner::GetChar(PRUnichar& aChar) {
  *  @param   
  *  @return  
  */
-PRInt32 CScanner::Peek(PRUnichar& aChar){
+PRInt32 CScanner::Peek(PRUnichar& aChar) {
   PRInt32 result=Eof();
   if(!result) {
     aChar=mBuffer[mOffset];