#167 ✓resolved
nickg

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

Reported by nickg | 2010-05-05 05:47:01 UTC

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 2010-05-05 06:34:49 UTC

    • 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