Subramanya Sastry has uploaded a new change for review.
Change subject: Refinement to 42d265a to strip non-nl WS around "|".
......................................................................
Refinement to 42d265a to strip non-nl WS around "|".
* Retaining white-space around "|" seems to introduce incorrect
<pre>'s in several cases which affects parse and RTing on the
following test pages:
- en:Barack_Obama
- en:Ta_Mok
- en:Knights_of_Classic_Lore
This experimental fix seems to eliminate parse and RT errors on
and possibly other pages.
* However, as noted in the FIXME in the tokenizer, the specific
fix seems incorrect since it will lead to incorrect parse in
the following two examples:
{{echo| }}a
{{echo|
}}a
The FIXME has more details and also lists a reduced test case
for analyzing this more thoroughly.
Change-Id: I099a7172205ec848bffdf4f66cc29d86b4ee11a6
---
M js/lib/pegTokenizer.pegjs.txt
1 file changed, 43 insertions(+), 17 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Parsoid
refs/changes/24/31724/1
--
To view, visit https://gerrit.wikimedia.org/r/31724
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I099a7172205ec848bffdf4f66cc29d86b4ee11a6
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Parsoid
Gerrit-Branch: master
Gerrit-Owner: Subramanya Sastry <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
