libretic
/
etherpad-lite
5
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Changeset: Throw on unexpected chars while iterating ops

This commit is contained in:
Richard Hansen 2021-10-21 03:17:28 -04:00
parent 657492e191
commit 86959f7ebc
1 changed files with 7 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
src/static/js/Changeset.js
View File

@ -197,20 +197,21 @@ exports.newLen = (cs) => exports.unpack(cs).newLen;
* @returns {OpIter} Operator iterator object.
*/
exports.opIterator = (opsStr) => {
const regex = /((?:\*[0-9a-z]+)*)(?:\|([0-9a-z]+))?([-+=])([0-9a-z]+)|\?|/g;
const regex = /((?:\*[0-9a-z]+)*)(?:\|([0-9a-z]+))?([-+=])([0-9a-z]+)|(.)/g;
const nextRegexMatch = () => {
const result = regex.exec(opsStr);
if (result[0] === '?') {
error('Hit error opcode in op stream');
}
if (!result) return null;
if (result[5] === '$') return null; // Start of the insert operation character bank.
if (result[5] != null) error(`invalid operation: ${opsStr.slice(regex.lastIndex - 1)}`);
return result;
};
let regexResult = nextRegexMatch();
const hasNext = () => regexResult && !!regexResult[0];
const next = (op = new Op()) => {
if (regexResult[0]) {
if (hasNext()) {
op.attribs = regexResult[1];
op.lines = exports.parseNum(regexResult[2] || '0');
op.opcode = regexResult[3];
@ -222,8 +223,6 @@ exports.opIterator = (opsStr) => {
return op;
};
const hasNext = () => !!(regexResult[0]);
return {
next,
hasNext,