wikimedia / parsoid / 3065 / 3

85%

master: 85%

Build Type

push

travis-ci

Committed by jenkins-bot

Commit Message

<a href="https://github.com/wikimedia/parsoid/commit/666d1f1d0">Use PHPUtils::lastItem() over end() in more places

Tim Starling has indicated in couple different places that end(..)
is not preferred and he had implemented a private version of lastItem
in the PEG grammar code whereas PHPUtils::lastItem was recommending
use of end(..).

In this patch, I moved the implementation from the grammar to PHPUtils
and replaced end(..) with PHPUtils::lastItem in a number of places in
the codebase. We should discuss whether we want to use this helper
everywhere.

Resolved a couple of PORT-FIXMEs in the bargain.

Change-Id: I</a><a class="double-link" href="https://github.com/wikimedia/parsoid/commit/837f2a980">837f2a980</a>

Run Details

9637 of 12147 branches covered (79.34%)

15086 of 17786 relevant lines covered (84.82%)

34117.22 hits per line