• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

aantron / markup.ml / 127
78%

Build:
DEFAULT BRANCH: master
Ran 08 May 2017 01:56AM UTC
Jobs 1
Files 21
Run time 1s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

pending completion
127

push

travis-ci

aantron
HTML: track the form element pointer

This is done for a (more-)correct behavior on an unmatched </form> tag.

The actual behavior is not quite right: when a form element is closed,
implied end tags are correctly generated for dd, dt, li, option,
optgroup, rb, rp, rt, rtc elements. If the element on the top of the
stack is not then the form element being closed, the specification says
to simply remove it from the stack.

This leaves the Markup.ml streaming parser unable to generate a proper
`End_element signal for it:

- If the signal is generated immediately, it will break nesting of the
signal stream, as the element on top of the stack has been opened, and
suddenly we are closing its ancestor form element.
- If the signal is not generated immediately, this will not accomplish
closing the form element, until its lingering presence triggers an
unrelated error, and it is closed haphazardly at a later point.

Note that this entire situation is a parse error to begin with, and
should not occur for correct markup.

This commit chooses to generate the signal immediately, but after first
closing all elements until the form element is reached, to preserve
nesting. This may result in early closing of (wrongly) unclosed elements
inside a form, in some error recovery scenarios.

Fixes aantron/lambda-soup#10.

24 of 24 new or added lines in 1 file covered. (100.0%)

3351 of 4293 relevant lines covered (78.06%)

347.27 hits per line

New Missed Lines in Diff

Lines Coverage ∆ File
6
100.0
src/markup_html_parser.ml
Jobs
ID Job ID Ran Files Coverage
4 127.4 (PPA=avsm/ocaml42+opam12 SWITCH=4.04.0+flambda DOCS=yes COVERALLS=yes) 08 May 2017 01:56AM UTC 0
78.06
Travis Job 127.4
Source Files on build 127
Detailed source file information is not available for this build.
  • Back to Repo
  • Travis Build #127
  • 0bf4f1b7 on github
  • Prev Build on master (#126)
  • Next Build on master (#128)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc