GWicke has submitted this change and it was merged.

Change subject: Updated TokenAndAttrCollector to handle multiple nested delims.
......................................................................


Updated TokenAndAttrCollector to handle multiple nested delims.

* The collector now collects all matching nested delimiters except
  for those that are balanced within a single-attribute.  It then
  merges pairs that are split across attributes of the same token
  because of parsing artefacts.  This in turn lets the nested
  attribute-expander call to deal with all matching pair of
  delimiter tokens.
* No change in parserTests.
* Bug fix in NoInclude handler.
* Several crashers now fixed:
  en:Nyc subway system (seem to have extra/unbalanced noinclude tags)
  en:Venn diagram
  en:Shomer Emunim (Hasidic dynasty)
  .. and others.
* To be completed: Switch over ExtensionContentCollector to use
  the TokenAndAttrCollector.

Change-Id: I52c430946f6cee0fe836d384ca9c7062b1c7ca74
---
M js/lib/ext.core.NoIncludeOnly.js
M js/lib/ext.util.TokenAndAttrCollector.js
M js/lib/pegTokenizer.pegjs.txt
3 files changed, 269 insertions(+), 60 deletions(-)

Approvals:
  GWicke: Verified; Looks good to me, approved


--
To view, visit https://gerrit.wikimedia.org/r/34217
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I52c430946f6cee0fe836d384ca9c7062b1c7ca74
Gerrit-PatchSet: 3
Gerrit-Project: mediawiki/extensions/Parsoid
Gerrit-Branch: master
Gerrit-Owner: Subramanya Sastry <[email protected]>
Gerrit-Reviewer: GWicke <[email protected]>
Gerrit-Reviewer: Subramanya Sastry <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to