Home / Browse / CyberNeko HTML Parser / Mail / Archive

CyberNeko HTML Parser

Email Archive: nekohtml-developer (read-only)

2007:	_Jan	_Feb	_Mar	_Apr	_May (1)	_Jun	_Jul (1)	_Aug	_Sep	_Oct (3)	_Nov (1)	_Dec (24)
2008:	_Jan (51)	_Feb (36)	_Mar (41)	_Apr (33)	_May (20)	_Jun (26)	_Jul (52)	_Aug (29)	_Sep (9)	_Oct (10)	_Nov (4)	_Dec (34)
2009:	_Jan (14)	_Feb (35)	_Mar (36)	_Apr (32)	_May (11)	_Jun (7)	_Jul (22)	_Aug (65)	_Sep (15)	_Oct (5)	_Nov (11)	_Dec (53)
2010:	_Jan (21)	_Feb (15)	_Mar (5)	_Apr	_May (13)	_Jun (8)	_Jul (3)	_Aug (2)	_Sep (2)	_Oct (4)	_Nov (4)	_Dec (3)
2011:	_Jan (2)	_Feb (2)	_Mar (7)	_Apr (1)	_May (2)	_Jun	_Jul (6)	_Aug (13)	_Sep (4)	_Oct (1)	_Nov	_Dec
2012:	_Jan (3)	_Feb (6)	_Mar (11)	_Apr (6)	_May (12)	_Jun (1)	_Jul (8)	_Aug (16)	_Sep (11)	_Oct (11)	_Nov (5)	_Dec (13)
2013:	_Jan (9)	_Feb (3)	_Mar (5)	_Apr (5)	_May (6)	_Jun	_Jul	_Aug	_Sep	_Oct	_Nov	_Dec


S	M	T	W	T	F	S
		1	2	3	4	5
			(3)		(2)
6	7	8	9	10	11	12
		(1)	(1)			(1)
13	14	15	16	17	18	19
	(2)
20	21	22	23	24	25	26
		(1)
27	28	29	30
(1)		(3)

[nekohtml-dev] [ nekohtml-Bugs-2870180 ] fragment context stack does not affect fSeenBodyElement

From: SourceForge.net <noreply@so...> - 2009-09-29 22:12

Bugs item #2870180, was opened at 2009-09-29 15:12
Message generated for change (Tracker Item Submitted) made by louiscryan
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=952178&aid=2870180&group_id=195122

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: tag-balancer
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Louis Ryan (louiscryan)
Assigned to: Nobody/Anonymous (nobody)
Summary: fragment context stack does not affect fSeenBodyElement

Initial Comment:
If the context stack for a fragment bound using http://cyberneko.org/html/properties/balance-tags/fragment-context-stack  contains a "body" tag Neko will still introduce a "head" tag if the first element in the content has HEAD as a preferred parent. This is caused by the fact that the boolean fSeenBodyElement is not affected by the context stack. Similarly fSeenRootElement and fSeenHeadElement are not affected when an "html" or "head" tag are present in the context stack

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=952178&aid=2870180&group_id=195122

[nekohtml-dev] [ nekohtml-Bugs-2854697 ] Stack Overflow Exception in HTMLTagBalancer.endElement

From: SourceForge.net <noreply@so...> - 2009-09-08 21:44

Bugs item #2854697, was opened at 2009-09-08 17:44
Message generated for change (Tracker Item Submitted) made by dgottlieb
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=952178&aid=2854697&group_id=195122

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: tag-balancer
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Daniel Gottlieb (dgottlieb)
Assigned to: Nobody/Anonymous (nobody)
Summary: Stack Overflow Exception in HTMLTagBalancer.endElement

Initial Comment:
The stack trace looks like:
java.lang.StackOverflowError
	at java.util.Arrays.copyOf(Arrays.java:2763)
	at java.util.Arrays.copyOf(Arrays.java:2734)
	at java.util.ArrayList.toArray(ArrayList.java:275)
	at java.util.ArrayList.<init>(ArrayList.java:131)
	at org.cyberneko.html.HTMLTagBalancer.consumeBufferedEndElements(HTMLTagBalancer.java:506)
	at org.cyberneko.html.HTMLTagBalancer.startElement(HTMLTagBalancer.java:589)
	at org.cyberneko.html.HTMLTagBalancer.forceStartElement(HTMLTagBalancer.java:758)
	at org.cyberneko.html.HTMLTagBalancer.startElement(HTMLTagBalancer.java:637)
	at org.cyberneko.html.HTMLTagBalancer.forceStartElement(HTMLTagBalancer.java:758)
	at org.cyberneko.html.HTMLTagBalancer.endElement(HTMLTagBalancer.java:999)
	at org.cyberneko.html.HTMLTagBalancer.endElement(HTMLTagBalancer.java:1000)
	at org.cyberneko.html.HTMLTagBalancer.endElement(HTMLTagBalancer.java:1000)
	at org.cyberneko.html.HTMLTagBalancer.endElement(HTMLTagBalancer.java:1000)
        (etc....)

I tried to minimize the amount of html necessary that recreates this and came up with:
<html>
  <noframes>
    <strike>text here is optional, but need a tag inside the noframes and the noframes must be outside the body</strike>
  </noframes>
  <body>
    <p>text here is optional, but it seems important that there is a paragraph tag inside the body</p>
  </body>
</html>


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=952178&aid=2854697&group_id=195122

[nekohtml-dev] [ nekohtml-Bugs-2854697 ] Stack Overflow Exception in HTMLTagBalancer.endElement

From: SourceForge.net <noreply@so...> - 2009-09-29 08:40

Bugs item #2854697, was opened at 2009-09-08 23:44
Message generated for change (Settings changed) made by mguillem
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=952178&aid=2854697&group_id=195122

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: tag-balancer
Group: None
>Status: Closed
>Resolution: Fixed
Priority: 5
Private: No
Submitted By: Daniel Gottlieb (dgottlieb)
>Assigned to: Marc Guillemot (mguillem)
Summary: Stack Overflow Exception in HTMLTagBalancer.endElement

Initial Comment:
The stack trace looks like:
java.lang.StackOverflowError
	at java.util.Arrays.copyOf(Arrays.java:2763)
	at java.util.Arrays.copyOf(Arrays.java:2734)
	at java.util.ArrayList.toArray(ArrayList.java:275)
	at java.util.ArrayList.<init>(ArrayList.java:131)
	at org.cyberneko.html.HTMLTagBalancer.consumeBufferedEndElements(HTMLTagBalancer.java:506)
	at org.cyberneko.html.HTMLTagBalancer.startElement(HTMLTagBalancer.java:589)
	at org.cyberneko.html.HTMLTagBalancer.forceStartElement(HTMLTagBalancer.java:758)
	at org.cyberneko.html.HTMLTagBalancer.startElement(HTMLTagBalancer.java:637)
	at org.cyberneko.html.HTMLTagBalancer.forceStartElement(HTMLTagBalancer.java:758)
	at org.cyberneko.html.HTMLTagBalancer.endElement(HTMLTagBalancer.java:999)
	at org.cyberneko.html.HTMLTagBalancer.endElement(HTMLTagBalancer.java:1000)
	at org.cyberneko.html.HTMLTagBalancer.endElement(HTMLTagBalancer.java:1000)
	at org.cyberneko.html.HTMLTagBalancer.endElement(HTMLTagBalancer.java:1000)
        (etc....)

I tried to minimize the amount of html necessary that recreates this and came up with:
<html>
  <noframes>
    <strike>text here is optional, but need a tag inside the noframes and the noframes must be outside the body</strike>
  </noframes>
  <body>
    <p>text here is optional, but it seems important that there is a paragraph tag inside the body</p>
  </body>
</html>


----------------------------------------------------------------------

>Comment By: Marc Guillemot (mguillem)
Date: 2009-09-29 10:40

Message:
Now fixed in SVN. Thanks for reporting with a minimal code example.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=952178&aid=2854697&group_id=195122

[nekohtml-dev] SF.net SVN: nekohtml:[259] trunk

From: <mguillem@us...> - 2009-09-02 08:18

Revision: 259
          http://nekohtml.svn.sourceforge.net/nekohtml/?rev=259&view=rev
Author:   mguillem
Date:     2009-09-02 08:18:30 +0000 (Wed, 02 Sep 2009)

Log Message:
-----------
preparing release 1.9.13

Modified Paths:
--------------
    trunk/build.xml
    trunk/doc/changes.html
    trunk/doc/index.html
    trunk/pom.xml

Modified: trunk/build.xml
===================================================================
--- trunk/build.xml	2009-08-25 08:45:24 UTC (rev 258)
+++ trunk/build.xml	2009-09-02 08:18:30 UTC (rev 259)
@@ -4,7 +4,7 @@
  <!-- PROPERTIES -->
  <property file='build-custom.properties' />
  
- <property name='version'   value='1.9.13-SNAPSHOT'/>
+ <property name='version'   value='1.9.13'/>
  <property name='name'      value='nekohtml'/>
  <property name='fullname'  value='${name}-${version}'/>
  <property name='Title'     value='NekoHTML'/>

Modified: trunk/doc/changes.html
===================================================================
--- trunk/doc/changes.html	2009-08-25 08:45:24 UTC (rev 258)
+++ trunk/doc/changes.html	2009-09-02 08:18:30 UTC (rev 259)
@@ -27,23 +27,25 @@
 
 <h2>Releases</h2>
 <dl>
- <dt>Future version
+ <dt>Version 1.9.13 (2 Sept 2009)
+     [<a href='http://downloads.sourceforge.net/nekohtml/nekohtml-1.9.13.zip'>zip</a>]
+     [<a href='http://downloads.sourceforge.net/nekohtml/nekohtml-1.9.13.tar.gz'>tgz</a>]
  <dd>A <code>A</code> tag should close opened <code>A</code> tag (fixes OutOfMemoryException reported in #2780607),
  not all entities with missing semicolon should be recognized as entities (different browsers behave differently therefore
  until we have something like "browser profiles", NekoHTML will only recognized incomplete entities with code less than 256),
- automatically add <code>TBODY</code> around <TR> nested directly within <code>TABLE</code>,
+ automatically add <code>TBODY</code> around <code>TR</code> nested directly within <code>TABLE</code>,
  <code>TD</code> and <code>TH</code> are container and should force closing of nested tags like <code>DIV</code>,
  don't generate incorrect warnings about missing closing <code>BODY</code> and <code>HTML</code> tags when feature <code>http://cyberneko.org/html/features/balance-tags/ignore-outside-content</code> is off,
  don't generate opening <code>TABLE</code> or <code>SELECT</code> tag when it's missing (therefore discard child nodes),
- added <font color='red'>experimental</font> tag balancer property <code>http://cyberneko.org/html/properties/balance-tags/fragment-context-stack</code> allowing to to specify the tag possibility to immediately evaluate a new input source during parsing,
- ignore whitespace that are direct children of <code>HTML</code>,
- org.cyberneko.html.filters.Writer wrongly change attribute value of some <code>META</code> tags (#2815779),
+ added <font color='red'>experimental</font> tag balancer property <code>http://cyberneko.org/html/properties/balance-tags/fragment-context-stack</code> allowing to to specify the current tag stack when evaluating a new input source during parsing,
+ ignore whitespaces that are direct children of <code>HTML</code>,
+ <code>org.cyberneko.html.filters.Writer</code> wrongly changes attribute value of some <code>META</code> tags (#2815779),
  use <code>http://www.w3.org/1999/xhtml</code> namespace for automatically inserted tags (#2799585),
  <code>DOMFragmentParser</code> ignores invalid processing instruction and attribute name (#2828553, #2828534),
  define latest Xerces version (2.9.1) as dependency in the published Maven POM,
  end of parent element forces closing of unknown elements (#2816091),
  handle new lines directly in HTMLScanner.scanCharacters to avoid multiple calls (patch provided by Sean Bridges, #2829319),
- fix ArrayIndexOutOfBoundsException occurring using <code>strip-comment-delims</code> feature (#2837555) 
+ fix ArrayIndexOutOfBoundsException occurring using <code>strip-comment-delims</code> feature (#2837555).
  
  
 

Modified: trunk/doc/index.html
===================================================================
--- trunk/doc/index.html	2009-08-25 08:45:24 UTC (rev 258)
+++ trunk/doc/index.html	2009-09-02 08:18:30 UTC (rev 259)
@@ -1,7 +1,7 @@
 <title>NekoHTML</title>
 <link rel=stylesheet type=text/css href=style.css>
 
-<h1>CyberNeko HTML Parser <sub>1.9.12</sub></h1>
+<h1>CyberNeko HTML Parser <sub>1.9.13</sub></h1>
 <div style="right: 10; top: 10; position: absolute">
 <a href="http://sourceforge.net/projects/nekohtml"><img src="http://sflogo.sourceforge.net/sflogo.php?group_id=195122&type=12" width="120" height="30" border="0" alt="Get NekoHTML at SourceForge.net. Fast, secure and Free Open Source software downloads" /></a>
 </div>
@@ -57,8 +57,8 @@
 following location:
 <ul>
 <li>NekoHTML 
-    [<a href='http://downloads.sourceforge.net/nekohtml/nekohtml-1.9.12.zip'>zip</a>]
-    [<a href='http://downloads.sourceforge.net/nekohtml/nekohtml-1.9.12.tar.gz'>tgz</a>] 
+    [<a href='http://downloads.sourceforge.net/nekohtml/nekohtml-1.9.13.zip'>zip</a>]
+    [<a href='http://downloads.sourceforge.net/nekohtml/nekohtml-1.9.13.tar.gz'>tgz</a>] 
 </ul>
 
 <h2>Requirements and Limitations</h2>

Modified: trunk/pom.xml
===================================================================
--- trunk/pom.xml	2009-08-25 08:45:24 UTC (rev 258)
+++ trunk/pom.xml	2009-09-02 08:18:30 UTC (rev 259)
@@ -3,7 +3,7 @@
   <groupId>net.sourceforge.nekohtml</groupId>
   <artifactId>nekohtml</artifactId>
   <name>Neko HTML</name>
-  <version>1.9.13-SNAPSHOT</version>
+  <version>1.9.13</version>
   <url>http://nekohtml.sourceforge.net/</url>
   <licenses>
     <license>


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.

[nekohtml-dev] SF.net SVN: nekohtml:[260] trunk

From: <mguillem@us...> - 2009-09-02 08:26

Revision: 260
          http://nekohtml.svn.sourceforge.net/nekohtml/?rev=260&view=rev
Author:   mguillem
Date:     2009-09-02 08:26:01 +0000 (Wed, 02 Sep 2009)

Log Message:
-----------
update copyright information, add myself

Modified Paths:
--------------
    trunk/build.xml
    trunk/doc/changes.html
    trunk/doc/faq.html
    trunk/doc/filters.html
    trunk/doc/index.html
    trunk/doc/settings.html
    trunk/doc/software.html
    trunk/doc/usage.html
    trunk/src/org/cyberneko/html/HTMLAugmentations.java
    trunk/src/org/cyberneko/html/HTMLComponent.java
    trunk/src/org/cyberneko/html/HTMLConfiguration.java
    trunk/src/org/cyberneko/html/HTMLElements.java
    trunk/src/org/cyberneko/html/HTMLEntities.java
    trunk/src/org/cyberneko/html/HTMLErrorReporter.java
    trunk/src/org/cyberneko/html/HTMLEventInfo.java
    trunk/src/org/cyberneko/html/HTMLScanner.java
    trunk/src/org/cyberneko/html/HTMLTagBalancer.java
    trunk/src/org/cyberneko/html/HTMLTagBalancingListener.java
    trunk/src/org/cyberneko/html/filters/DefaultFilter.java
    trunk/src/org/cyberneko/html/filters/ElementRemover.java
    trunk/src/org/cyberneko/html/filters/Identity.java
    trunk/src/org/cyberneko/html/filters/Writer.java
    trunk/src/org/cyberneko/html/parsers/DOMFragmentParser.java
    trunk/src/org/cyberneko/html/parsers/DOMParser.java
    trunk/src/org/cyberneko/html/parsers/SAXParser.java
    trunk/src/org/cyberneko/html/xercesbridge/XercesBridge.java
    trunk/src/org/cyberneko/html/xercesbridge/XercesBridge_2_0.java
    trunk/src/org/cyberneko/html/xercesbridge/XercesBridge_2_1.java
    trunk/src/org/cyberneko/html/xercesbridge/XercesBridge_2_2.java
    trunk/src/org/cyberneko/html/xercesbridge/XercesBridge_2_3.java
    trunk/src/sample/HTMLSAXParser.java
    trunk/src/sample/RemoveElements.java
    trunk/src/sample/Script.java
    trunk/src/sample/TestHTMLDOM.java
    trunk/src/sample/TestHTMLDOMFragment.java
    trunk/test/java/org/cyberneko/html/CanonicalTest.java
    trunk/test/java/org/cyberneko/html/Writer.java

Modified: trunk/build.xml
===================================================================
--- trunk/build.xml	2009-09-02 08:18:30 UTC (rev 259)
+++ trunk/build.xml	2009-09-02 08:26:01 UTC (rev 260)
@@ -10,7 +10,7 @@
  <property name='Title'     value='NekoHTML'/>
  <property name='FullTitle' value='CyberNeko HTML Parser'/>
  <property name='Name'      value='${Title} ${version}'/>
- <property name='author'    value='Andy Clark'/>
+ <property name='author'    value='Andy Clark, Marc Guillemot'/>
  <property name='copyright' value='(C) Copyright 2002-2008, ${author}. All rights reserved.'/>
  <property name='URL'       value='http://nekohtml.sourceforge.net/index.html'/>
  

Modified: trunk/doc/changes.html
===================================================================
--- trunk/doc/changes.html	2009-09-02 08:18:30 UTC (rev 259)
+++ trunk/doc/changes.html	2009-09-02 08:26:01 UTC (rev 260)
@@ -553,5 +553,5 @@
 </dl>
 
 <div class='copyright'>
-(C) Copyright 2002-2008, Andy Clark. All rights reserved.
+(C) Copyright 2002-2009, Andy Clark, Marc Guillemot. All rights reserved.
 </div>
\ No newline at end of file

Modified: trunk/doc/faq.html
===================================================================
--- trunk/doc/faq.html	2009-09-02 08:18:30 UTC (rev 259)
+++ trunk/doc/faq.html	2009-09-02 08:26:01 UTC (rev 260)
@@ -361,5 +361,5 @@
 </pre>
 
 <div class='copyright'>
-(C) Copyright 2002-2008, Andy Clark. All rights reserved.
+(C) Copyright 2002-2009, Andy Clark, Marc Guillemot. All rights reserved.
 </div>

Modified: trunk/doc/filters.html
===================================================================
--- trunk/doc/filters.html	2009-09-02 08:18:30 UTC (rev 259)
+++ trunk/doc/filters.html	2009-09-02 08:26:01 UTC (rev 260)
@@ -371,5 +371,5 @@
 </pre>
 
 <div class='copyright'>
-(C) Copyright 2002-2008, Andy Clark. All rights reserved.
+(C) Copyright 2002-2009, Andy Clark, Marc Guillemot. All rights reserved.
 </div>
\ No newline at end of file

Modified: trunk/doc/index.html
===================================================================
--- trunk/doc/index.html	2009-09-02 08:18:30 UTC (rev 259)
+++ trunk/doc/index.html	2009-09-02 08:26:01 UTC (rev 260)
@@ -114,5 +114,5 @@
 a bug</a>.
 
 <div class='copyright'>
-(C) Copyright 2002-2008, Andy Clark. All rights reserved.
+(C) Copyright 2002-2009, Andy Clark, Marc Guillemot. All rights reserved.
 </div>

Modified: trunk/doc/settings.html
===================================================================
--- trunk/doc/settings.html	2009-09-02 08:18:30 UTC (rev 259)
+++ trunk/doc/settings.html	2009-09-02 08:26:01 UTC (rev 260)
@@ -452,5 +452,5 @@
 </table>
 
 <div class='copyright'>
-(C) Copyright 2002-2008, Andy Clark. All rights reserved.
+(C) Copyright 2002-2009, Andy Clark, Marc Guillemot. All rights reserved.
 </div>

Modified: trunk/doc/software.html
===================================================================
--- trunk/doc/software.html	2009-09-02 08:18:30 UTC (rev 259)
+++ trunk/doc/software.html	2009-09-02 08:26:01 UTC (rev 260)
@@ -206,5 +206,5 @@
 mailing list</a>.
 
 <div class='copyright'>
-(C) Copyright 2002-2008, Andy Clark. All rights reserved.
+(C) Copyright 2002-2009, Andy Clark, Marc Guillemot. All rights reserved.
 </div>
\ No newline at end of file

Modified: trunk/doc/usage.html
===================================================================
--- trunk/doc/usage.html	2009-09-02 08:18:30 UTC (rev 259)
+++ trunk/doc/usage.html	2009-09-02 08:26:01 UTC (rev 260)
@@ -210,5 +210,5 @@
 This source code is included in the <code>src/sample/</code> directory.
 
 <div class='copyright'>
-(C) Copyright 2002-2008, Andy Clark. All rights reserved.
+(C) Copyright 2002-2009, Andy Clark, Marc Guillemot. All rights reserved.
 </div>
\ No newline at end of file

Modified: trunk/src/org/cyberneko/html/HTMLAugmentations.java
===================================================================
--- trunk/src/org/cyberneko/html/HTMLAugmentations.java	2009-09-02 08:18:30 UTC (rev 259)
+++ trunk/src/org/cyberneko/html/HTMLAugmentations.java	2009-09-02 08:26:01 UTC (rev 260)
@@ -1,5 +1,5 @@
 /* 
- * Copyright 2004-2008 Andy Clark
+ * Copyright 2004-2008 Andy Clark, Marc Guillemot
  * 
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.

Modified: trunk/src/org/cyberneko/html/HTMLComponent.java
===================================================================
--- trunk/src/org/cyberneko/html/HTMLComponent.java	2009-09-02 08:18:30 UTC (rev 259)
+++ trunk/src/org/cyberneko/html/HTMLComponent.java	2009-09-02 08:26:01 UTC (rev 260)
@@ -1,5 +1,5 @@
 /* 
- * Copyright 2002-2008 Andy Clark
+ * Copyright 2004-2008 Andy Clark, Marc Guillemot
  * 
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.

Modified: trunk/src/org/cyberneko/html/HTMLConfiguration.java
===================================================================
--- trunk/src/org/cyberneko/html/HTMLConfiguration.java	2009-09-02 08:18:30 UTC (rev 259)
+++ trunk/src/org/cyberneko/html/HTMLConfiguration.java	2009-09-02 08:26:01 UTC (rev 260)
@@ -1,5 +1,5 @@
 /* 
- * Copyright 2002-2008 Andy Clark
+ * Copyright 2002-2009 Andy Clark, Marc Guillemot
  * 
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.

Modified: trunk/src/org/cyberneko/html/HTMLElements.java
===================================================================
--- trunk/src/org/cyberneko/html/HTMLElements.java	2009-09-02 08:18:30 UTC (rev 259)
+++ trunk/src/org/cyberneko/html/HTMLElements.java	2009-09-02 08:26:01 UTC (rev 260)
@@ -1,5 +1,5 @@
 /* 
- * Copyright 2002-2008 Andy Clark
+ * Copyright 2002-2009 Andy Clark, Marc Guillemot
  * 
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.

Modified: trunk/src/org/cyberneko/html/HTMLEntities.java
===================================================================
--- trunk/src/org/cyberneko/html/HTMLEntities.java	2009-09-02 08:18:30 UTC (rev 259)
+++ trunk/src/org/cyberneko/html/HTMLEntities.java	2009-09-02 08:26:01 UTC (rev 260)
@@ -1,5 +1,5 @@
 /* 
- * Copyright 2002-2008 Andy Clark
+ * Copyright 2002-2009 Andy Clark, Marc Guillemot
  * 
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.

Modified: trunk/src/org/cyberneko/html/HTMLErrorReporter.java
===================================================================
--- trunk/src/org/cyberneko/html/HTMLErrorReporter.java	2009-09-02 08:18:30 UTC (rev 259)
+++ trunk/src/org/cyberneko/html/HTMLErrorReporter.java	2009-09-02 08:26:01 UTC (rev 260)
@@ -1,5 +1,5 @@
 /* 
- * Copyright 2002-2008 Andy Clark
+ * Copyright 2002-2009 Andy Clark, Marc Guillemot
  * 
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.

Modified: trunk/src/org/cyberneko/html/HTMLEventInfo.java
===================================================================
--- trunk/src/org/cyberneko/html/HTMLEventInfo.java	2009-09-02 08:18:30 UTC (rev 259)
+++ trunk/src/org/cyberneko/html/HTMLEventInfo.java	2009-09-02 08:26:01 UTC (rev 260)
@@ -1,5 +1,5 @@
 /* 
- * Copyright 2002-2008 Andy Clark
+ * Copyright 2002-2009 Andy Clark, Marc Guillemot
  * 
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.

Modified: trunk/src/org/cyberneko/html/HTMLScanner.java
===================================================================
--- trunk/src/org/cyberneko/html/HTMLScanner.java	2009-09-02 08:18:30 UTC (rev 259)
+++ trunk/src/org/cyberneko/html/HTMLScanner.java	2009-09-02 08:26:01 UTC (rev 260)
@@ -1,5 +1,5 @@
 /* 
- * Copyright 2002-2008 Andy Clark
+ * Copyright 2002-2009 Andy Clark, Marc Guillemot
  * 
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.

Modified: trunk/src/org/cyberneko/html/HTMLTagBalancer.java
===================================================================
--- trunk/src/org/cyberneko/html/HTMLTagBalancer.java	2009-09-02 08:18:30 UTC (rev 259)
+++ trunk/src/org/cyberneko/html/HTMLTagBalancer.java	2009-09-02 08:26:01 UTC (rev 260)
@@ -1,5 +1,5 @@
 /* 
- * Copyright 2002-2008 Andy Clark
+ * Copyright 2002-2009 Andy Clark, Marc Guillemot
  * 
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.

Modified: trunk/src/org/cyberneko/html/HTMLTagBalancingListener.java
===================================================================
--- trunk/src/org/cyberneko/html/HTMLTagBalancingListener.java	2009-09-02 08:18:30 UTC (rev 259)
+++ trunk/src/org/cyberneko/html/HTMLTagBalancingListener.java	2009-09-02 08:26:01 UTC (rev 260)
@@ -1,5 +1,5 @@
 /* 
- * Copyright 2002-2008 Andy Clark
+ * Copyright 2002-2009 Andy Clark, Marc Guillemot
  * 
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.

Modified: trunk/src/org/cyberneko/html/filters/DefaultFilter.java
===================================================================
--- trunk/src/org/cyberneko/html/filters/DefaultFilter.java	2009-09-02 08:18:30 UTC (rev 259)
+++ trunk/src/org/cyberneko/html/filters/DefaultFilter.java	2009-09-02 08:26:01 UTC (rev 260)
@@ -1,5 +1,5 @@
 /* 
- * Copyright 2002-2008 Andy Clark
+ * Copyright 2002-2009 Andy Clark, Marc Guillemot
  * 
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.

Modified: trunk/src/org/cyberneko/html/filters/ElementRemover.java
===================================================================
--- trunk/src/org/cyberneko/html/filters/ElementRemover.java	2009-09-02 08:18:30 UTC (rev 259)
+++ trunk/src/org/cyberneko/html/filters/ElementRemover.java	2009-09-02 08:26:01 UTC (rev 260)
@@ -1,5 +1,5 @@
 /* 
- * Copyright 2002-2008 Andy Clark
+ * Copyright 2002-2009 Andy Clark, Marc Guillemot
  * 
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.

Modified: trunk/src/org/cyberneko/html/filters/Identity.java
===================================================================
--- trunk/src/org/cyberneko/html/filters/Identity.java	2009-09-02 08:18:30 UTC (rev 259)
+++ trunk/src/org/cyberneko/html/filters/Identity.java	2009-09-02 08:26:01 UTC (rev 260)
@@ -1,5 +1,5 @@
 /* 
- * Copyright 2002-2008 Andy Clark
+ * Copyright 2002-2009 Andy Clark, Marc Guillemot
  * 
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.

Modified: trunk/src/org/cyberneko/html/filters/Writer.java
===================================================================
--- trunk/src/org/cyberneko/html/filters/Writer.java	2009-09-02 08:18:30 UTC (rev 259)
+++ trunk/src/org/cyberneko/html/filters/Writer.java	2009-09-02 08:26:01 UTC (rev 260)
@@ -1,5 +1,5 @@
 /* 
- * Copyright 2002-2008 Andy Clark
+ * Copyright 2002-2009 Andy Clark, Marc Guillemot
  * 
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.

Modified: trunk/src/org/cyberneko/html/parsers/DOMFragmentParser.java
===================================================================
--- trunk/src/org/cyberneko/html/parsers/DOMFragmentParser.java	2009-09-02 08:18:30 UTC (rev 259)
+++ trunk/src/org/cyberneko/html/parsers/DOMFragmentParser.java	2009-09-02 08:26:01 UTC (rev 260)
@@ -1,5 +1,5 @@
 /* 
- * Copyright 2002-2008 Andy Clark
+ * Copyright 2002-2009 Andy Clark, Marc Guillemot
  * 
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.

Modified: trunk/src/org/cyberneko/html/parsers/DOMParser.java
===================================================================
--- trunk/src/org/cyberneko/html/parsers/DOMParser.java	2009-09-02 08:18:30 UTC (rev 259)
+++ trunk/src/org/cyberneko/html/parsers/DOMParser.java	2009-09-02 08:26:01 UTC (rev 260)
@@ -1,5 +1,5 @@
 /* 
- * Copyright 2002-2008 Andy Clark
+ * Copyright 2002-2009 Andy Clark, Marc Guillemot
  * 
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.

Modified: trunk/src/org/cyberneko/html/parsers/SAXParser.java
===================================================================
--- trunk/src/org/cyberneko/html/parsers/SAXParser.java	2009-09-02 08:18:30 UTC (rev 259)
+++ trunk/src/org/cyberneko/html/parsers/SAXParser.java	2009-09-02 08:26:01 UTC (rev 260)
@@ -1,5 +1,5 @@
 /* 
- * Copyright 2002-2008 Andy Clark
+ * Copyright 2002-2009 Andy Clark, Marc Guillemot
  * 
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.

Modified: trunk/src/org/cyberneko/html/xercesbridge/XercesBridge.java
===================================================================
--- trunk/src/org/cyberneko/html/xercesbridge/XercesBridge.java	2009-09-02 08:18:30 UTC (rev 259)
+++ trunk/src/org/cyberneko/html/xercesbridge/XercesBridge.java	2009-09-02 08:26:01 UTC (rev 260)
@@ -1,3 +1,18 @@
+/* 
+ * Copyright Marc Guillemot
+ * 
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
 package org.cyberneko.html.xercesbridge;
 
 import org.apache.xerces.xni.Augmentations;

Modified: trunk/src/org/cyberneko/html/xercesbridge/XercesBridge_2_0.java
===================================================================
--- trunk/src/org/cyberneko/html/xercesbridge/XercesBridge_2_0.java	2009-09-02 08:18:30 UTC (rev 259)
+++ trunk/src/org/cyberneko/html/xercesbridge/XercesBridge_2_0.java	2009-09-02 08:26:01 UTC (rev 260)
@@ -1,3 +1,18 @@
+/* 
+ * Copyright Marc Guillemot
+ * 
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
 package org.cyberneko.html.xercesbridge;
 
 import java.lang.reflect.InvocationTargetException;

Modified: trunk/src/org/cyberneko/html/xercesbridge/XercesBridge_2_1.java
===================================================================
--- trunk/src/org/cyberneko/html/xercesbridge/XercesBridge_2_1.java	2009-09-02 08:18:30 UTC (rev 259)
+++ trunk/src/org/cyberneko/html/xercesbridge/XercesBridge_2_1.java	2009-09-02 08:26:01 UTC (rev 260)
@@ -1,3 +1,18 @@
+/* 
+ * Copyright Marc Guillemot
+ * 
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
 package org.cyberneko.html.xercesbridge;
 
 import org.apache.xerces.impl.Version;

Modified: trunk/src/org/cyberneko/html/xercesbridge/XercesBridge_2_2.java
===================================================================
--- trunk/src/org/cyberneko/html/xercesbridge/XercesBridge_2_2.java	2009-09-02 08:18:30 UTC (rev 259)
+++ trunk/src/org/cyberneko/html/xercesbridge/XercesBridge_2_2.java	2009-09-02 08:26:01 UTC (rev 260)
@@ -1,3 +1,18 @@
+/* 
+ * Copyright Marc Guillemot
+ * 
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
 package org.cyberneko.html.xercesbridge;
 
 import org.apache.xerces.impl.Version;

Modified: trunk/src/org/cyberneko/html/xercesbridge/XercesBridge_2_3.java
===================================================================
--- trunk/src/org/cyberneko/html/xercesbridge/XercesBridge_2_3.java	2009-09-02 08:18:30 UTC (rev 259)
+++ trunk/src/org/cyberneko/html/xercesbridge/XercesBridge_2_3.java	2009-09-02 08:26:01 UTC (rev 260)
@@ -1,3 +1,18 @@
+/* 
+ * Copyright Marc Guillemot
+ * 
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
 package org.cyberneko.html.xercesbridge;
 
 import org.apache.xerces.xni.NamespaceContext;

Modified: trunk/src/sample/HTMLSAXParser.java
===================================================================
--- trunk/src/sample/HTMLSAXParser.java	2009-09-02 08:18:30 UTC (rev 259)
+++ trunk/src/sample/HTMLSAXParser.java	2009-09-02 08:26:01 UTC (rev 260)
@@ -1,5 +1,5 @@
 /* 
- * Copyright 2002-2008 Andy Clark
+ * Copyright 2002-2009 Andy Clark, Marc Guillemot
  * 
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.

Modified: trunk/src/sample/RemoveElements.java
===================================================================
--- trunk/src/sample/RemoveElements.java	2009-09-02 08:18:30 UTC (rev 259)
+++ trunk/src/sample/RemoveElements.java	2009-09-02 08:26:01 UTC (rev 260)
@@ -1,5 +1,5 @@
 /* 
- * Copyright 2002-2008 Andy Clark
+ * Copyright 2002-2009 Andy Clark, Marc Guillemot
  * 
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.

Modified: trunk/src/sample/Script.java
===================================================================
--- trunk/src/sample/Script.java	2009-09-02 08:18:30 UTC (rev 259)
+++ trunk/src/sample/Script.java	2009-09-02 08:26:01 UTC (rev 260)
@@ -1,5 +1,5 @@
 /* 
- * Copyright 2002-2008 Andy Clark
+ * Copyright 2002-2009 Andy Clark, Marc Guillemot
  * 
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.

Modified: trunk/src/sample/TestHTMLDOM.java
===================================================================
--- trunk/src/sample/TestHTMLDOM.java	2009-09-02 08:18:30 UTC (rev 259)
+++ trunk/src/sample/TestHTMLDOM.java	2009-09-02 08:26:01 UTC (rev 260)
@@ -1,5 +1,5 @@
 /* 
- * Copyright 2002-2008 Andy Clark
+ * Copyright 2002-2009 Andy Clark, Marc Guillemot
  * 
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.

Modified: trunk/src/sample/TestHTMLDOMFragment.java
===================================================================
--- trunk/src/sample/TestHTMLDOMFragment.java	2009-09-02 08:18:30 UTC (rev 259)
+++ trunk/src/sample/TestHTMLDOMFragment.java	2009-09-02 08:26:01 UTC (rev 260)
@@ -1,5 +1,5 @@
 /* 
- * Copyright 2002-2008 Andy Clark
+ * Copyright 2002-2009 Andy Clark, Marc Guillemot
  * 
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.

Modified: trunk/test/java/org/cyberneko/html/CanonicalTest.java
===================================================================
--- trunk/test/java/org/cyberneko/html/CanonicalTest.java	2009-09-02 08:18:30 UTC (rev 259)
+++ trunk/test/java/org/cyberneko/html/CanonicalTest.java	2009-09-02 08:26:01 UTC (rev 260)
@@ -1,5 +1,5 @@
 /* 
- * Copyright 2002-2008 Andy Clark
+ * Copyright 2002-2009 Andy Clark, Marc Guillemot
  * 
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.

Modified: trunk/test/java/org/cyberneko/html/Writer.java
===================================================================
--- trunk/test/java/org/cyberneko/html/Writer.java	2009-09-02 08:18:30 UTC (rev 259)
+++ trunk/test/java/org/cyberneko/html/Writer.java	2009-09-02 08:26:01 UTC (rev 260)
@@ -1,5 +1,5 @@
 /* 
- * Copyright 2002-2008 Andy Clark
+ * Copyright 2002-2009 Andy Clark, Marc Guillemot
  * 
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.

[nekohtml-dev] SF.net SVN: nekohtml:[262] trunk

From: <mguillem@us...> - 2009-09-29 08:39

Revision: 262
          http://nekohtml.svn.sourceforge.net/nekohtml/?rev=262&view=rev
Author:   mguillem
Date:     2009-09-29 08:39:08 +0000 (Tue, 29 Sep 2009)

Log Message:
-----------
Don't parse body of NOFRAMES (fixes StackOverflowError reported in #2854697)

Issue #2854697

Modified Paths:
--------------
    trunk/doc/changes.html
    trunk/src/org/cyberneko/html/HTMLElements.java
    trunk/src/org/cyberneko/html/HTMLScanner.java

Added Paths:
-----------
    trunk/data/noxxx/
    trunk/data/noxxx/test-noframes.html
    trunk/data/noxxx/test-noframes.html.canonical
    trunk/data/noxxx/test-noscript-parseit.html
    trunk/data/noxxx/test-noscript-parseit.html.canonical
    trunk/data/noxxx/test-noscript.html
    trunk/data/noxxx/test-noscript.html.canonical
    trunk/data/noxxx/test-noscript.html.settings

Removed Paths:
-------------
    trunk/data/canonical/test-noscript-parseit.html
    trunk/data/canonical/test-noscript.html
    trunk/data/test-noscript-parseit.html
    trunk/data/test-noscript.html
    trunk/data/test-noscript.html.settings

Deleted: trunk/data/canonical/test-noscript-parseit.html
===================================================================
--- trunk/data/canonical/test-noscript-parseit.html	2009-09-02 08:49:11 UTC (rev 261)
+++ trunk/data/canonical/test-noscript-parseit.html	2009-09-29 08:39:08 UTC (rev 262)
@@ -1,14 +0,0 @@
-(HTML
-(HEAD
-)HEAD
-(BODY
-(NOSCRIPT
-(DIV
-"hello 
-(SPAN
-"world
-)SPAN
-)DIV
-)NOSCRIPT
-)BODY
-)HTML
\ No newline at end of file

Deleted: trunk/data/canonical/test-noscript.html
===================================================================
--- trunk/data/canonical/test-noscript.html	2009-09-02 08:49:11 UTC (rev 261)
+++ trunk/data/canonical/test-noscript.html	2009-09-29 08:39:08 UTC (rev 262)
@@ -1,9 +0,0 @@
-(HTML
-(HEAD
-)HEAD
-(BODY
-(NOSCRIPT
-"<div>hello <span>world</span>
-)NOSCRIPT
-)BODY
-)HTML
\ No newline at end of file

Added: trunk/data/noxxx/test-noframes.html
===================================================================
--- trunk/data/noxxx/test-noframes.html	                        (rev 0)
+++ trunk/data/noxxx/test-noframes.html	2009-09-29 08:39:08 UTC (rev 262)
@@ -0,0 +1,8 @@
+<html>
+<noframes>
+<p>hello
+</noframes>
+<body>
+<p>hello again
+</body>
+</html>
\ No newline at end of file

Added: trunk/data/noxxx/test-noframes.html.canonical
===================================================================
--- trunk/data/noxxx/test-noframes.html.canonical	                        (rev 0)
+++ trunk/data/noxxx/test-noframes.html.canonical	2009-09-29 08:39:08 UTC (rev 262)
@@ -0,0 +1,13 @@
+(HTML
+(NOFRAMES
+"\n<p>hello\n
+)NOFRAMES
+(HEAD
+)HEAD
+(BODY
+"\n
+(P
+"hello again\n
+)P
+)BODY
+)HTML
\ No newline at end of file

Copied: trunk/data/noxxx/test-noscript-parseit.html (from rev 261, trunk/data/test-noscript-parseit.html)
===================================================================
--- trunk/data/noxxx/test-noscript-parseit.html	                        (rev 0)
+++ trunk/data/noxxx/test-noscript-parseit.html	2009-09-29 08:39:08 UTC (rev 262)
@@ -0,0 +1 @@
+<noscript><div>hello <span>world</span></noscript>
\ No newline at end of file

Copied: trunk/data/noxxx/test-noscript-parseit.html.canonical (from rev 261, trunk/data/canonical/test-noscript-parseit.html)
===================================================================
--- trunk/data/noxxx/test-noscript-parseit.html.canonical	                        (rev 0)
+++ trunk/data/noxxx/test-noscript-parseit.html.canonical	2009-09-29 08:39:08 UTC (rev 262)
@@ -0,0 +1,14 @@
+(HTML
+(HEAD
+)HEAD
+(BODY
+(NOSCRIPT
+(DIV
+"hello 
+(SPAN
+"world
+)SPAN
+)DIV
+)NOSCRIPT
+)BODY
+)HTML
\ No newline at end of file

Copied: trunk/data/noxxx/test-noscript.html (from rev 261, trunk/data/test-noscript.html)
===================================================================
--- trunk/data/noxxx/test-noscript.html	                        (rev 0)
+++ trunk/data/noxxx/test-noscript.html	2009-09-29 08:39:08 UTC (rev 262)
@@ -0,0 +1 @@
+<noscript><div>hello <span>world</span></noscript>
\ No newline at end of file

Copied: trunk/data/noxxx/test-noscript.html.canonical (from rev 261, trunk/data/canonical/test-noscript.html)
===================================================================
--- trunk/data/noxxx/test-noscript.html.canonical	                        (rev 0)
+++ trunk/data/noxxx/test-noscript.html.canonical	2009-09-29 08:39:08 UTC (rev 262)
@@ -0,0 +1,9 @@
+(HTML
+(HEAD
+)HEAD
+(BODY
+(NOSCRIPT
+"<div>hello <span>world</span>
+)NOSCRIPT
+)BODY
+)HTML
\ No newline at end of file

Copied: trunk/data/noxxx/test-noscript.html.settings (from rev 261, trunk/data/test-noscript.html.settings)
===================================================================
--- trunk/data/noxxx/test-noscript.html.settings	                        (rev 0)
+++ trunk/data/noxxx/test-noscript.html.settings	2009-09-29 08:39:08 UTC (rev 262)
@@ -0,0 +1 @@
+feature http://cyberneko.org/html/features/parse-noscript-content false

Deleted: trunk/data/test-noscript-parseit.html
===================================================================
--- trunk/data/test-noscript-parseit.html	2009-09-02 08:49:11 UTC (rev 261)
+++ trunk/data/test-noscript-parseit.html	2009-09-29 08:39:08 UTC (rev 262)
@@ -1 +0,0 @@
-<noscript><div>hello <span>world</span></noscript>
\ No newline at end of file

Deleted: trunk/data/test-noscript.html
===================================================================
--- trunk/data/test-noscript.html	2009-09-02 08:49:11 UTC (rev 261)
+++ trunk/data/test-noscript.html	2009-09-29 08:39:08 UTC (rev 262)
@@ -1 +0,0 @@
-<noscript><div>hello <span>world</span></noscript>
\ No newline at end of file

Deleted: trunk/data/test-noscript.html.settings
===================================================================
--- trunk/data/test-noscript.html.settings	2009-09-02 08:49:11 UTC (rev 261)
+++ trunk/data/test-noscript.html.settings	2009-09-29 08:39:08 UTC (rev 262)
@@ -1 +0,0 @@
-feature http://cyberneko.org/html/features/parse-noscript-content false

Modified: trunk/doc/changes.html
===================================================================
--- trunk/doc/changes.html	2009-09-02 08:49:11 UTC (rev 261)
+++ trunk/doc/changes.html	2009-09-29 08:39:08 UTC (rev 262)
@@ -27,6 +27,11 @@
 
 <h2>Releases</h2>
 <dl>
+ <dt>Future version</dt>
+ <dd>
+ Don't parse body of <code>NOFRAMES</code> (fixes StackOverflowError reported in #2854697)
+ </dd>
+ 
  <dt>Version 1.9.13 (2 Sept 2009)
      [<a href='http://downloads.sourceforge.net/nekohtml/nekohtml-1.9.13.zip'>zip</a>]
      [<a href='http://downloads.sourceforge.net/nekohtml/nekohtml-1.9.13.tar.gz'>tgz</a>]

Modified: trunk/src/org/cyberneko/html/HTMLElements.java
===================================================================
--- trunk/src/org/cyberneko/html/HTMLElements.java	2009-09-02 08:49:11 UTC (rev 261)
+++ trunk/src/org/cyberneko/html/HTMLElements.java	2009-09-29 08:39:08 UTC (rev 262)
@@ -349,7 +349,7 @@
             // NOEMBED
             new Element(NOEMBED, "NOEMBED", 0, BODY, null),
             // NOFRAMES - - (BODY) -(NOFRAMES)
-            new Element(NOFRAMES, "NOFRAMES", 0, FRAMESET, null),
+            new Element(NOFRAMES, "NOFRAMES", 0, null, null),
             // NOLAYER
             new Element(NOLAYER, "NOLAYER", 0, BODY, null),
             // NOSCRIPT - - (%block;)+

Modified: trunk/src/org/cyberneko/html/HTMLScanner.java
===================================================================
--- trunk/src/org/cyberneko/html/HTMLScanner.java	2009-09-02 08:49:11 UTC (rev 261)
+++ trunk/src/org/cyberneko/html/HTMLScanner.java	2009-09-29 08:39:08 UTC (rev 262)
@@ -401,6 +401,9 @@
     /** Parse noscript content. */
     protected boolean fParseNoScriptContent;
 
+    /** Parse noframes content. */
+    protected boolean fParseNoFramesContent;
+
     // properties
 
     /** Modify HTML element names. */
@@ -2018,8 +2021,11 @@
                                 	scanScriptContent();
                                 }
                                 else if (!fParseNoScriptContent && "noscript".equalsIgnoreCase(ename)) {
-                                	scanNoScriptContent();
+                                	scanNoXxxContent("noscript");
                                 }
+                                else if (!fParseNoFramesContent && "noframes".equalsIgnoreCase(ename)) {
+                                	scanNoXxxContent("noframes");
+                                }
                                 else if (ename != null && !fSingleBoolean[0] 
                                     && HTMLElements.getElement(ename).isSpecial() 
                                     && (!ename.equalsIgnoreCase("TITLE") || isEnded(ename))) {
@@ -2086,10 +2092,12 @@
         /**
          * Scans the content of <noscript>: it doesn't get parsed but is considered as plain text
          * when feature {@link HTMLScanner#PARSE_NOSCRIPT_CONTENT} is set to false.
+         * @param the tag for which content is scanned (one of "noscript" or "noframes")
          * @throws IOException
          */
-        private void scanNoScriptContent() throws IOException {
+        private void scanNoXxxContent(final String tagName) throws IOException {
         	final XMLStringBuffer buffer = new XMLStringBuffer();
+        	final String end = "/" + tagName;
         	
             while (true) {
                 int c = read();
@@ -2098,7 +2106,7 @@
                 }
                 if (c == '<') {
                 	final String next = nextContent(10) + " ";
-                	if (next.length() >= 10 && "/noscript".equalsIgnoreCase(next.substring(0, 9))
+                	if (next.length() >= 10 && end.equalsIgnoreCase(next.substring(0, end.length()))
             			&& ('>' == next.charAt(9) || Character.isWhitespace(next.charAt(9)))) {
 	                    fCurrentEntity.offset--;
                             fCurrentEntity.characterOffset--;


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.

[nekohtml-dev] [ nekohtml-Bugs-2868326 ] Returns after </body> should be ignored

From: SourceForge.net <noreply@so...> - 2009-09-27 19:30

Bugs item #2868326, was opened at 2009-09-27 12:30
Message generated for change (Tracker Item Submitted) made by kkrugler
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=952178&aid=2868326&group_id=195122

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: scanner
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Ken Krugler (kkrugler)
Assigned to: Nobody/Anonymous (nobody)
Summary: Returns after </body> should be ignored

Initial Comment:
Currently a document ending in </body></html> will output two extra newlines before the closing </body> tag. I assume this is to handle invalid HTML, where text is included after the terminating body or html tags.

It's a minor issue, but it made my comparison with other parsers fail badly the first time, since I almost always got extra content (trailing newlines) that didn't match.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=952178&aid=2868326&group_id=195122

[nekohtml-dev] [ nekohtml-Bugs-2864276 ] Stack overflow on document with two <BODY>s

From: SourceForge.net <noreply@so...> - 2009-09-22 12:24

Bugs item #2864276, was opened at 2009-09-22 14:24
Message generated for change (Tracker Item Submitted) made by aaronkaplan
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=952178&aid=2864276&group_id=195122

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: tag-balancer
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Aaron Kaplan (aaronkaplan)
Assigned to: Nobody/Anonymous (nobody)
Summary: Stack overflow on document with two <BODY>s

Initial Comment:
Parsing the attached document causes a stack overflow.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=952178&aid=2864276&group_id=195122

[nekohtml-dev] [ nekohtml-Support Requests-2857500 ] remove invalid namespaces from XML fragment

From: SourceForge.net <noreply@so...> - 2009-09-12 13:26

Support Requests item #2857500, was opened at 2009-09-12 13:26
Message generated for change (Tracker Item Submitted) made by salk31
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=952179&aid=2857500&group_id=195122

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
Status: Open
Priority: 5
Private: No
Submitted By: Sam Hough (salk31)
Assigned to: Nobody/Anonymous (nobody)
Summary: remove invalid namespaces from XML fragment

Initial Comment:
I'm using the SAXParser to clean up HTML fragments provided by the user, cut and pasted into editor with hideous Word rubbish, and pass on, via SAX, to Xalan.

I've tried various combinations of properties but I can't get it to remove invalid namespaces or create synthetic declarations (are these allowed on non-root elements?) using Purifier filter.

What set of features/properties should I be using? At the moment I'm using a ContentHandler to strip them but this seems overkill.

I'm using 1.9.13, Java 1.6.0_0-b11 and Xerces 2.6.2

Many thanks. nekohtml has been cleaning HTML for me for years. I'm just being fussy now ;)

Cheers

Sam

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=952179&aid=2857500&group_id=195122

[nekohtml-dev] [ nekohtml-Support Requests-2857500 ] remove invalid namespaces from XML fragment

From: SourceForge.net <noreply@so...> - 2009-09-14 04:31

Support Requests item #2857500, was opened at 2009-09-12 15:26
Message generated for change (Comment added) made by mguillem
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=952179&aid=2857500&group_id=195122

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
>Status: Pending
Priority: 5
Private: No
Submitted By: Sam Hough (salk31)
Assigned to: Nobody/Anonymous (nobody)
Summary: remove invalid namespaces from XML fragment

Initial Comment:
I'm using the SAXParser to clean up HTML fragments provided by the user, cut and pasted into editor with hideous Word rubbish, and pass on, via SAX, to Xalan.

I've tried various combinations of properties but I can't get it to remove invalid namespaces or create synthetic declarations (are these allowed on non-root elements?) using Purifier filter.

What set of features/properties should I be using? At the moment I'm using a ContentHandler to strip them but this seems overkill.

I'm using 1.9.13, Java 1.6.0_0-b11 and Xerces 2.6.2

Many thanks. nekohtml has been cleaning HTML for me for years. I'm just being fussy now ;)

Cheers

Sam

----------------------------------------------------------------------

>Comment By: Marc Guillemot (mguillem)
Date: 2009-09-14 06:31

Message:
Can you provide an example of your dirty html code as well as what you
expect as cleaned result?

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=952179&aid=2857500&group_id=195122

[nekohtml-dev] [ nekohtml-Support Requests-2857500 ] remove invalid namespaces from XML fragment

From: SourceForge.net <noreply@so...> - 2009-09-14 17:53

Support Requests item #2857500, was opened at 2009-09-12 13:26
Message generated for change (Comment added) made by salk31
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=952179&aid=2857500&group_id=195122

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
>Status: Open
Priority: 5
Private: No
Submitted By: Sam Hough (salk31)
Assigned to: Nobody/Anonymous (nobody)
Summary: remove invalid namespaces from XML fragment

Initial Comment:
I'm using the SAXParser to clean up HTML fragments provided by the user, cut and pasted into editor with hideous Word rubbish, and pass on, via SAX, to Xalan.

I've tried various combinations of properties but I can't get it to remove invalid namespaces or create synthetic declarations (are these allowed on non-root elements?) using Purifier filter.

What set of features/properties should I be using? At the moment I'm using a ContentHandler to strip them but this seems overkill.

I'm using 1.9.13, Java 1.6.0_0-b11 and Xerces 2.6.2

Many thanks. nekohtml has been cleaning HTML for me for years. I'm just being fussy now ;)

Cheers

Sam

----------------------------------------------------------------------

>Comment By: Sam Hough (salk31)
Date: 2009-09-14 17:53

Message:
I've attached a unit test for this. I'm happy to make match nekohtml coding
conventions and reattach as a patch if useful.

Anyway, the not very consistent tests I have are:
    public void testElementWithNamespace() throws Exception {
        doTest("<strong><o:p></o:p>X</strong>",
"<strong><p/>X</strong>");
    }

    public void testRemoveAttibuteWithNamespace() throws Exception {
        doTest("<p m:bogus=\"x\">fred</p>", "<p>fred</p>");
    }

Mainly I just want to avoid Xalan blowing up due to undeclared
namespaces.

Many thanks

Sam

----------------------------------------------------------------------

Comment By: Marc Guillemot (mguillem)
Date: 2009-09-14 04:31

Message:
Can you provide an example of your dirty html code as well as what you
expect as cleaned result?

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=952179&aid=2857500&group_id=195122

[nekohtml-dev] [ nekohtml-Bugs-2851952 ] HTML element should not be added for document fragments

From: SourceForge.net <noreply@so...> - 2009-09-04 23:16

Bugs item #2851952, was opened at 2009-09-04 16:16
Message generated for change (Settings changed) made by dhauge496
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=952178&aid=2851952&group_id=195122

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: tag-balancer
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Doug Hauge (dhauge496)
Assigned to: Nobody/Anonymous (nobody)
>Summary: HTML element should not be added for document fragments

Initial Comment:
When parsing document fragments (http://cyberneko.org/html/features/balance-tags/document-fragment   set to 'true'), the load balancer should not add HTML, HEAD, or BODY elements, but it does. It turns

<div></div>

into

<html><head></head><body><div></div></body></html>

This worked correctly in 1.9.11.


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=952178&aid=2851952&group_id=195122

[nekohtml-dev] [ nekohtml-Bugs-2851952 ] HTML element should not be added for document fragments

From: SourceForge.net <noreply@so...> - 2009-09-09 00:49

Bugs item #2851952, was opened at 2009-09-04 16:16
Message generated for change (Comment added) made by bmargolin
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=952178&aid=2851952&group_id=195122

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: tag-balancer
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Doug Hauge (dhauge496)
Assigned to: Nobody/Anonymous (nobody)
Summary: HTML element should not be added for document fragments

Initial Comment:
When parsing document fragments (http://cyberneko.org/html/features/balance-tags/document-fragment   set to 'true'), the load balancer should not add HTML, HEAD, or BODY elements, but it does. It turns

<div></div>

into

<html><head></head><body><div></div></body></html>

This worked correctly in 1.9.11.


----------------------------------------------------------------------

Comment By: Ben Margolin (bmargolin)
Date: 2009-09-08 17:49

Message:
+1 to this getting fixed. Does wacky stuff like

foo <b>bar</b>

being parsed into

foo <html><head /><body><b>bar</b></body></html>

which is way wrong, but used to work properly.

As a workaround I was able to 'get back' the older behavior by doing

       
parser.setFeature("http://cyberneko.org/html/features/balance-tags",
false);
       
parser.setFeature("http://cyberneko.org/html/features/balance-tags/document-fragment",
false);

before parsing, but this doesn't seem like it should be necessary.


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=952178&aid=2851952&group_id=195122

[nekohtml-dev] [ nekohtml-Bugs-2851952 ] HTML elemen should not be added for document fragments

From: SourceForge.net <noreply@so...> - 2009-09-04 23:16

Bugs item #2851952, was opened at 2009-09-04 16:16
Message generated for change (Tracker Item Submitted) made by dhauge496
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=952178&aid=2851952&group_id=195122

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: tag-balancer
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Doug Hauge (dhauge496)
Assigned to: Nobody/Anonymous (nobody)
Summary: HTML elemen should not be added for document fragments

Initial Comment:
When parsing document fragments (http://cyberneko.org/html/features/balance-tags/document-fragment   set to 'true'), the load balancer should not add HTML, HEAD, or BODY elements, but it does. It turns

<div></div>

into

<html><head></head><body><div></div></body></html>

This worked correctly in 1.9.11.


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=952178&aid=2851952&group_id=195122

[nekohtml-dev] SF.net SVN: nekohtml:[261] branches/nekohtml-1.9.13/

From: <mguillem@us...> - 2009-09-02 08:49

Revision: 261
          http://nekohtml.svn.sourceforge.net/nekohtml/?rev=261&view=rev
Author:   mguillem
Date:     2009-09-02 08:49:11 +0000 (Wed, 02 Sep 2009)

Log Message:
-----------
Release 1.9.13

Added Paths:
-----------
    branches/nekohtml-1.9.13/


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.