Merge remote-tracking branch 'origin/main' into dave/win/2024-03-11

src/bun.js/api/BunObject.zig

@@ -3181,10 +3181,20 @@ pub export fn Bun__escapeHTML16(globalObject: *JSC.JSGlobalObject, input_value:
         .allocated => |escaped_html| {
             if (comptime Environment.allow_assert) {
                 // assert that re-encoding the string produces the same result
+                const allocated = strings.toUTF16AllocForReal(
+                    bun.default_allocator,
+                    strings.toUTF8Alloc(bun.default_allocator, escaped_html) catch unreachable,
+                    false,
+                    false,
+                ) catch {
+                    globalObject.throwOutOfMemory();
+                    return .zero;
+                };
+                defer bun.default_allocator.free(allocated);
                 std.debug.assert(
                     std.mem.eql(
                         u16,
-                        (strings.toUTF16Alloc(bun.default_allocator, strings.toUTF8Alloc(bun.default_allocator, escaped_html) catch unreachable, false, false) catch unreachable).?,
+                        allocated,
                         escaped_html,
                     ),
                 );

src/bun.js/api/bun/process.zig

@@ -400,10 +400,6 @@ pub const Process = struct {
         } else if (Environment.isWindows) {
             switch (this.poller) {
                 .uv => |*process| {
-                    if (comptime !Environment.isWindows) {
-                        unreachable;
-                    }
-
                     if (process.isClosed()) {
                         this.poller = .{ .detached = {} };
                     } else if (!process.isClosing()) {