Use the new writeNetString api

qti3e · Aug 10, 2019 · f2ae932 · f2ae932
1 parent 00ba16d
commit f2ae932
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 10 deletions.
diff --git a/jsc/src/scope.ts b/jsc/src/scope.ts
@@ -24,19 +24,13 @@ export class Scope {
   addFunction(name: string, id: number): void {
     this.buffer.writeUint16(++this.num, 0);
     this.buffer.put(1);
-    // TODO(qti3e) This is wrong... name.length is not the
-    // same as number of bytes.
-    this.buffer.writeUint16(name.length);
-    this.buffer.writeString(name);
+    this.buffer.writeNetString16(name);
     this.buffer.writeUint16(id);
   }
 
   addVariable(name: string): void {
     this.buffer.writeUint16(++this.num, 0);
     this.buffer.put(0);
-    // TODO(qti3e) This is wrong... name.length is not the
-    // same as number of bytes.
-    this.buffer.writeUint16(name.length);
-    this.buffer.writeString(name);
+    this.buffer.writeNetString16(name);
   }
 }
diff --git a/jsc/src/writer.ts b/jsc/src/writer.ts
@@ -51,8 +51,7 @@ export class Writer {
 
     if (constantPoolData) {
       const index = this.constantPool.getCursor();
-      this.constantPool.writeInt16(constantPoolData.length);
-      this.constantPool.writeString(constantPoolData);
+      this.constantPool.writeNetString16(constantPoolData);
       this.codeSection.writeUint32(index);
     }
   }