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

wikimedia / parsoid / 776 / 2
85%
master: 85%

Build:
DEFAULT BRANCH: master
Ran 11 Nov 2015 07:56AM UTC
Files 73
Run time 11s
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

11 Nov 2015 07:56AM UTC coverage: 84.015% (+0.02%) from 84.0%
776.2

push

travis-ci

subbuss
T118367: Kill dead code + fix bad perf in pathological scenarios.

This code was added in fd0aefc3 (Aug 3, 2012) in the very
first draft of the template encapsulation code when nothing
but the very outlines of this technique had been dreamt up.

This code was seemlingly added to deal with fostered content from
tables (suprise!). But, given how our template wrapping works,
about tags are only assigned to the start/end meta tags and
nothing else in between. So, there was no way that a table
could exist with an about-id found on transclusion meta tags.

You can verify this by examining code in tt/TemplateHandler.js
and/or by running this:
echo "{{echo|a\n\nb\n\ncr\n\nd}}" | node parse --dump dom:pre-encap
and noticing that none of the p-tags there have an about id.

So, this code has always (or at least since almost the very
beginning) been dead, and only served to make parsing slow in
pathological cases as in the bug report.

With this dead code removed, that page parses in 17s.

Change-Id: I056169ada

12640 of 15045 relevant lines covered (84.01%)

21538.64 hits per line

Source Files on job 776.2
  • Tree
  • List 0
  • Changed 24
  • Source Changed 1
  • Coverage Changed 24
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Build 776
  • Travis Job 776.2
  • 392e25eb on github
  • Prev Job for on master (#775.2)
  • Next Job for on master (#777.2)
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