#167 ✓resolved
nickg

htmlparser can use too much recursion if "useTimeouts" is false

Reported by nickg | May 5th, 2010 @ 05:47 AM

in pump function
in HtmlParser.java
in env-js/htmlparser/gwt2/src/nu/validator/htmlparser/gwt

where "useTimeouts" is false

uses recursion (calling pump) for what appears to be no reason. This should be converted to iterative. I bumped into the case where recursion got so deep, the interp bailed. oops.

Comments and changes to this ticket

  • nickg

    nickg May 5th, 2010 @ 06:34 AM

    • State changed from “open” to “resolved”

    hmm, I thought this would be hard. not so. Commited... but this code isn't hooked into envjs yet.

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.

New-ticket Create new ticket

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile »

a javascript browser environment

People watching this ticket

Pages