From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from out.smtp-auth.no-ip.com (smtp-auth.no-ip.com [8.23.224.61]) by hurricane.the-brannons.com (Postfix) with ESMTPS id AE4C8789BC for ; Wed, 23 Mar 2016 23:38:07 -0700 (PDT) X-No-IP: carhart.net@noip-smtp X-Report-Spam-To: abuse@no-ip.com Received: from carhart.net (unknown [99.52.200.227]) (Authenticated sender: carhart.net@noip-smtp) by smtp-auth.no-ip.com (Postfix) with ESMTPA id 24268400960 for ; Wed, 23 Mar 2016 23:38:21 -0700 (PDT) Received: from kevc (carhart.net [192.168.1.179]) by carhart.net (8.13.8/8.13.8) with ESMTP id u2O6cKmp017341; Wed, 23 Mar 2016 23:38:20 -0700 To: Edbrowse-dev@lists.the-brannons.com From: Kevin Carhart Reply-to: Kevin Carhart User-Agent: edbrowse/3.5.4.2 Date: Wed, 23 Mar 2016 23:38:20 -0700 Message-ID: <20160223233820.kevin@carhart.net > Mime-Version: 1.0 Content-Type: text/plain Content-Transfer-Encoding: 7bit Subject: [Edbrowse-dev] [patch] TidyEscapeScripts X-BeenThere: edbrowse-dev@lists.the-brannons.com X-Mailman-Version: 2.1.21 Precedence: list List-Id: Edbrowse Development List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 24 Mar 2016 06:38:07 -0000 Thanks for the remarks about frames. I'll revisit that shortly but let's get tidy issue 348 out of our hair. Geoff said that his suggestion for how to implement this is what is shown in the following patch. It sets the new tidy option to 'no' by default in edbrowse, meaning don't parse scripts by default, meaning do make the change by default. Whereas when you compile tidy, the new option is set to 'yes' by default, meaning, leave things as they have been for continuity. So the feature is there but latent until you make the setting. Thanks Kevin >>From c4b58703036dd77db3ce14562ff89e10383b0664 Mon Sep 17 00:00:00 2001 From: Kevin Carhart Date: Wed, 23 Mar 2016 23:12:45 -0700 Subject: [PATCH] sets new tidy opt 'EscapeScripts' to 'no' by default --- src/html-tidy.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/html-tidy.c b/src/html-tidy.c index 7c086ad..d121c21 100644 --- a/src/html-tidy.c +++ b/src/html-tidy.c @@ -119,6 +119,10 @@ void html2nodes(const char *htmltext, bool startpage) tidyOptSetInt(tdoc, TidyBodyOnly, yes); tidySetReportFilter(tdoc, tidyErrorHandler); // tidySetReportFilter(tdoc, tidyReportFilter); + + // the following tidyOptSetBool implements + // a fix for https://github.com/htacg/tidy-html5/issues/348 + tidyOptSetBool( tdoc, TidyEscapeScripts, no ); tidySetCharEncoding(tdoc, (cons_utf8 ? "utf8" : "latin1")); -- 1.8.3.2