Page MenuHomePhabricator

whitespace handling regression in 8915e64
Closed, ResolvedPublic

Description

8915e64 seems to have introduced a few small round-trip regressions. Most of the ar: regressions in http://parsoid.wmflabs.org:8001/regressions/between/9b52271863840018fd7770b2e4fa57b404075c1a/028bb94cbc8b11fc4ab07af499d840c4c26f7d34/0 seem to be caused by it.

test case:
git checkout 8915e64

node roundtrip-test.js --prefix ar 'كابياء_بهلوانية'

SUMMARY:
Semantic differences : 3

Syntactic differences: 0

git checkout 3d6d0723f9 (branch point for 8915e64)

node roundtrip-test.js --prefix ar 'كابياء_بهلوانية'

SUMMARY:
Semantic differences : 0

Syntactic differences: 3


Version: unspecified
Severity: normal

Details

Reference
bz55902

Event Timeline

bzimport raised the priority of this task from to High.Nov 22 2014, 2:17 AM
bzimport set Reference to bz55902.

Change 91048 had a related patch set uploaded by Subramanya Sastry:
[Bug 55902]: <BR> special case in makeSeparator

https://gerrit.wikimedia.org/r/91048

Verified that this fixes the test case in the bug description.

Change 91048 merged by jenkins-bot:
[Bug 55902]: <BR> special case in makeSeparator

https://gerrit.wikimedia.org/r/91048