From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: None (mailfrom) identity=mailfrom; client-ip=8.23.224.62; helo=out.smtp-auth.no-ip.com; envelope-from=kevin@carhart.net; receiver= Received: from out.smtp-auth.no-ip.com (smtp-auth.no-ip.com [8.23.224.62]) by hurricane.the-brannons.com (Postfix) with ESMTPS id 3735477C83 for ; Thu, 24 Aug 2017 20:58:34 -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 AED893CA; Thu, 24 Aug 2017 20:59:01 -0700 (PDT) Received: from carhart.net (localhost [127.0.0.1]) by carhart.net (8.13.8/8.13.8) with ESMTP id v7P3x0oB026674; Thu, 24 Aug 2017 20:59:00 -0700 Received: from localhost (kevin@localhost) by carhart.net (8.13.8/8.13.8/Submit) with ESMTP id v7P3wxG9026671; Thu, 24 Aug 2017 20:59:00 -0700 Date: Thu, 24 Aug 2017 20:58:59 -0700 (PDT) From: Kevin Carhart To: Karl Dahlke cc: Edbrowse-dev@lists.the-brannons.com In-Reply-To: <20170724061500.eklhad@comcast.net> Message-ID: References: <20170724061500.eklhad@comcast.net> User-Agent: Alpine 2.03 (LRH 1266 2009-07-14) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed Subject: Re: [Edbrowse-dev] other bugs X-BeenThere: edbrowse-dev@lists.the-brannons.com X-Mailman-Version: 2.1.24 Precedence: list List-Id: Edbrowse Development List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 25 Aug 2017 03:58:34 -0000 Thank you for discovering these. Too bad but I guess like you said... > acid test hunts for corner cases so not surprising it uncovers bugs. Exactly... it's going to exercise a lot since that's what it's built to do. One piece of good news is that a few days ago when we added something and then I said not "it works," but "we now have the pleasure of having exposed the next problem. It is frustrating but actually a good thing," I temporarily stayed mum on what it was we had exposed. What we had exposed was that for me at least, edbrowse was now hanging for an unrelated reason, way off in test 43. Should not be misinterpreted as a failure even though the outward manifestation is "it's hanging, this is bad." So the punchline is that as of the last couple days with the CSS code, edbrowse no longer hangs. I don't know the cause because test 43 is way off, but the cessation of hanging is nice. Was it a helpful format to send cloneable .git links for the two projects? Because, in case it was, here is the git link for the javascript DOM that I have mentioned: git clone https://github.com/thatcher/env-js.git If you clone it, go to src. And then there is a css folder. Maybe there is something in there worth cribbing or referring to that you (KD or others) would like to know about. Kevin > I already wrote about the tidy bug, not giving us the empty nodes, which test 0 expects to be there, and so I guess every other browser puts them there. > I sent this question to Geoff. > But turning to css, there are more problems. > The parser that Kevin found for me doesn't parse comments corectly. > If a comment contains a selector it is treated as a selector, not a comment, and it's all screwed up from there. > I reported this to jotform, no response yet. > Ok this one seems like an easy work around. > I don't need the comments preserved for any reason, so just strip them out with a preprocessing regexp. > > css_string = css_string.replace(/\/\*(.|\s)*?\*\//g, ''); > > Yeah I know, it looks like line noise in the modem, but it's right, I'm pretty sure, and works on the modest test strings that I give it, > but it doesn't work on the style block that is part of acid test 0. > RangeError: regexp executor recursion limit > I used the nongreedy modifier to the wild card, so there really shouldn't be any recursion at all, so I don't get it. > I sent this to one of the duktape developers. > > With all these issues hanging, I'm rather in a holding pattern as per the acid tests. > > Karl Dahlke > -------- Kevin Carhart * 415 225 5306 * The Ten Ninety Nihilists