Changeset: Replace appendATextToAssembler() with a generator

Author: rhansen

CHANGELOG.md

@@ -39,6 +39,8 @@
       function instead.
     * The `mergingOpAssembler()` function is deprecated; use the new
       `squashOps()` generator function with `serializeOps()` instead.
+    * The `appendATextToAssembler()` function is deprecated; use the new
+      `opsFromAText()` generator function instead.
 
 ### Notable enhancements
 

src/node/db/Pad.js

@@ -490,7 +490,7 @@ Pad.prototype.copyPadWithoutHistory = async function (destinationID, force) {
 
   // based on Changeset.makeSplice
   const assem = Changeset.smartOpAssembler();
-  Changeset.appendATextToAssembler(oldAText, assem);
+  for (const op of Changeset.opsFromAText(oldAText)) assem.append(op);
   assem.endDocument();
 
   // although we have instantiated the newPad with '\n', an additional '\n' is

src/static/js/Changeset.js

@@ -1806,16 +1806,15 @@ exports.copyAText = (atext1, atext2) => {
 };
 
 /**
- * Append the set of operations from atext to an assembler.
- *
- * @param {AText} atext -
- * @param assem - Assembler like SmartOpAssembler TODO add desc
+ * Convert AText to a series of operations.
+ * @param {AText} atext - The AText to convert.
+ * @yields {Op}
  */
-exports.appendATextToAssembler = (atext, assem) => {
+exports.opsFromAText = function* (atext) {
   // intentionally skips last newline char of atext
   let lastOp = null;
   for (const op of new exports.OpIter(atext.attribs)) {
-    if (lastOp != null) assem.append(lastOp);
+    if (lastOp != null) yield lastOp;
     lastOp = op;
   }
   if (lastOp == null) return;
@@ -1828,11 +1827,24 @@ exports.appendATextToAssembler = (atext, assem) => {
     const lastLineLength = atext.text.length - nextToLastNewlineEnd - 1;
     lastOp.lines--;
     lastOp.chars -= (lastLineLength + 1);
-    assem.append(lastOp);
+    yield copyOp(lastOp);
     lastOp.lines = 0;
     lastOp.chars = lastLineLength;
   }
-  if (lastOp.chars) assem.append(lastOp);
+  if (lastOp.chars) yield lastOp;
+};
+
+/**
+ * Append the set of operations from atext to an assembler.
+ *
+ * @deprecated Use `opsFromAText` instead.
+ * @param {AText} atext -
+ * @param assem - Assembler like SmartOpAssembler TODO add desc
+ */
+exports.appendATextToAssembler = (atext, assem) => {
+  warnDeprecated(
+      'Changeset.appendATextToAssembler() is deprecated; use Changeset.opsFromAText() instead');
+  for (const op of exports.opsFromAText(atext)) assem.append(op);
 };
 
 /**

src/static/js/ace2_inner.js

@@ -538,7 +538,7 @@ function Ace2Inner(editorInfo, cssManagers) {
     o.chars = lastLineLength;
     o.lines = 0;
     assem.append(o);
-    Changeset.appendATextToAssembler(atext, assem);
+    for (const op of Changeset.opsFromAText(atext)) assem.append(op);
     const newLen = oldLen + assem.getLengthChange();
     const changeset = Changeset.checkRep(
         Changeset.pack(oldLen, newLen, assem.toString(), atext.text.slice(0, -1)));

src/tests/frontend/specs/easysync.js

@@ -741,7 +741,7 @@ describe('easysync', function () {
   const testAppendATextToAssembler = (testId, atext, correctOps) => {
     it(`testAppendATextToAssembler#${testId}`, async function () {
       const assem = Changeset.smartOpAssembler();
-      Changeset.appendATextToAssembler(atext, assem);
+      for (const op of Changeset.opsFromAText(atext)) assem.append(op);
       expect(assem.toString()).to.equal(correctOps);
     });
   };