Subramanya Sastry has uploaded a new change for review.
Change subject: Additional serializer fixes.
......................................................................
Additional serializer fixes.
* Removed "\n" emitting from p-handlers so this can be rerouted
via the separator callback.
* Added an 'emitsNL' flag when a handler needs to emit a newline
unconditionally -- this is then routed through the emitNLs
callback.
* All separators between nodes are now emitted via a single
callback -- rather than one for endsLine for prev-node and
startsLine for current-node. This is done by routing all
separator emits via emitSepChunk and not emitting buffered
separators from emitSeparators. Instead, when a handler starts
a line, this sep is merged with the buffered separator and
emitted.
* TODO: Verify that all cases are covered.
* TODO: br emits an id('\n'), but this is probably not a separator.
* No change in parser tests.
Change-Id: I0de931cf663990196e2d8a9672cc055659031d2b
---
M js/lib/mediawiki.WikitextSerializer.js
1 file changed, 42 insertions(+), 41 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Parsoid
refs/changes/86/42586/1
--
To view, visit https://gerrit.wikimedia.org/r/42586
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I0de931cf663990196e2d8a9672cc055659031d2b
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
