perf(runtime/ops): use tuple instead of object

Author: CyanChanges

ext/os/README.md

@@ -35,10 +35,8 @@ This crate implements OS specific APIs for Deno
 
 `cpu_usage`
 
-| Target family | Syscall                                        | Description |
-| ------------- |------------------------------------------------| ----------- |
-| Linux         | getrusage                                      | -           |
-| Windows       | `processthreadsapi::GetProcessTimes`           | -           |
-| macOS         | getrusage                                      | -           |
-
-
+| Target family | Syscall                              | Description |
+| ------------- | ------------------------------------ | ----------- |
+| Linux         | getrusage                            | -           |
+| Windows       | `processthreadsapi::GetProcessTimes` | -           |
+| macOS         | getrusage                            | -           |

ext/os/lib.rs

@@ -422,20 +422,11 @@ fn op_uid(state: &mut OpState) -> Result<Option<u32>, PermissionCheckError> {
   Ok(None)
 }
 
-#[derive(Serialize)]
-struct CpuUsage {
-  system: usize,
-  user: usize,
-}
-
 #[op2]
 #[serde]
-fn op_runtime_cpu_usage() -> CpuUsage {
+fn op_runtime_cpu_usage() -> (usize, usize) {
   let (sys, user) = get_cpu_usage();
-  CpuUsage {
-    system: sys.as_micros() as usize,
-    user: user.as_micros() as usize,
-  }
+  (sys.as_micros() as usize, user.as_micros() as usize)
 }
 
 #[cfg(unix)]
@@ -536,27 +527,22 @@ fn get_cpu_usage() -> (std::time::Duration, std::time::Duration) {
   Default::default()
 }
 
-// HeapStats stores values from a isolate.get_heap_statistics() call
-#[derive(Serialize)]
-#[serde(rename_all = "camelCase")]
-struct MemoryUsage {
-  rss: usize,
-  heap_total: usize,
-  heap_used: usize,
-  external: usize,
-}
-
 #[op2]
 #[serde]
-fn op_runtime_memory_usage(scope: &mut v8::HandleScope) -> MemoryUsage {
+fn op_runtime_memory_usage(
+  scope: &mut v8::HandleScope,
+) -> (usize, usize, usize, usize) {
   let mut s = v8::HeapStatistics::default();
   scope.get_heap_statistics(&mut s);
-  MemoryUsage {
-    rss: rss(),
-    heap_total: s.total_heap_size(),
-    heap_used: s.used_heap_size(),
-    external: s.external_memory(),
-  }
+
+  let (rss, heap_total, heap_used, external) = (
+    rss(),
+    s.total_heap_size(),
+    s.used_heap_size(),
+    s.external_memory(),
+  );
+
+  (rss, heap_total, heap_used, external)
 }
 
 #[cfg(any(target_os = "android", target_os = "linux"))]

runtime/js/90_deno_ns.js

@@ -66,7 +66,7 @@ const denoNs = {
   memoryUsage: () => {
     const { 0: rss, 1: heapTotal, 2: heapUsed, 3: external } =
       op_runtime_memory_usage();
-    return { rss, heapTotal, heapUsed, external };
+    return { rss, heapTotal, heapUsed, external, arrayBuffers: 0 };
   },
   mkdirSync: fs.mkdirSync,
   mkdir: fs.mkdir,