You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@shindig.apache.org by jo...@apache.org on 2010/04/07 04:04:16 UTC
svn commit: r931404 -
/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/parse/GadgetHtmlParser.java
Author: johnh
Date: Wed Apr 7 02:04:16 2010
New Revision: 931404
URL: http://svn.apache.org/viewvc?rev=931404&view=rev
Log:
Remove final from parseDom and parseFragment functions to allow creating different parsers.
For example a router between multiple parsers.
Patch provided by Ziv Horesh.
Modified:
shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/parse/GadgetHtmlParser.java
Modified: shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/parse/GadgetHtmlParser.java
URL: http://svn.apache.org/viewvc/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/parse/GadgetHtmlParser.java?rev=931404&r1=931403&r2=931404&view=diff
==============================================================================
--- shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/parse/GadgetHtmlParser.java (original)
+++ shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/parse/GadgetHtmlParser.java Wed Apr 7 02:04:16 2010
@@ -89,7 +89,7 @@ public abstract class GadgetHtmlParser {
return normalized.contains("<!DOCTYPE") || normalized.contains("<HTML");
}
- public final Document parseDom(String source) throws GadgetException {
+ public Document parseDom(String source) throws GadgetException {
Document document = null;
String key = null;
// Avoid checksum overhead if we arent caching
@@ -233,7 +233,7 @@ public abstract class GadgetHtmlParser {
* @param result Node to append results to
* @throws GadgetException
*/
- public final void parseFragment(String source, Node result) throws GadgetException {
+ public void parseFragment(String source, Node result) throws GadgetException {
boolean shouldCache = shouldCache();
String key = null;
if (shouldCache) {