[j8] Always use the J8 printer for 'json write'

oilshell · Dec 27, 2023 · bcb8521 · bcb8521
1 parent ca30ed0
commit bcb8521
Showing 1 changed file with 7 additions and 20 deletions.
diff --git a/builtin/json_ysh.py b/builtin/json_ysh.py
@@ -87,26 +87,13 @@ def Run(self, cmd_val):
 
             #log('json write indent %d', indent)
 
-            # TODO: We should use the new printer unconditionally
-            if self.is_j8:
-                buf = mylib.BufWriter()
-                self.printer.Print(val, buf, indent)
-                self.stdout_.write(buf.getvalue())
-                self.stdout_.write('\n')
-            else:
-                if mylib.PYTHON:
-                    #if 0:
-                    obj = cpython._ValueToPyObj(val)
-
-                    j = yajl.dumps(obj, indent=indent)
-                    sys.stdout.write(j)
-                    if extra_newline:
-                        sys.stdout.write('\n')
-                else:
-                    buf = mylib.BufWriter()
-                    self.printer.Print(val, buf, indent)
-                    self.stdout_.write(buf.getvalue())
-                    self.stdout_.write('\n')
+            buf = mylib.BufWriter()
+
+            # TODO: Call PrintMessage() vs. PrintJsonmessage()
+            self.printer.Print(val, buf, indent)
+
+            self.stdout_.write(buf.getvalue())
+            self.stdout_.write('\n')
 
         elif action == 'read':
             attrs = flag_spec.Parse('json_read', arg_r)