Skip to content

Commit 9640bbe

Browse files
kateinoigakukunkateinoigakukun
committed
BridgeJS: Restore new SwiftClass-style constructor by using Object.create 
1 parent d8523db commit 9640bbe

File tree

5 files changed

+38
-44
lines changed

5 files changed

+38
-44
lines changed

‎Plugins/BridgeJS/Sources/BridgeJSLink/BridgeJSLink.swift

Lines changed: 12 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -43,18 +43,16 @@ struct BridgeJSLink {
4343
let swiftHeapObjectClassJs = """
4444
/// Represents a Swift heap object like a class instance or an actor instance.
4545
class SwiftHeapObject {
46-
static __construct(ptr, deinit) {
47-
return new SwiftHeapObject(ptr, deinit);
48-
}
49-
50-
constructor(pointer, deinit) {
51-
this.pointer = pointer;
52-
this.hasReleased = false;
53-
this.deinit = deinit;
54-
this.registry = new FinalizationRegistry((pointer) => {
46+
static __wrap(pointer, deinit, prototype) {
47+
const obj = Object.create(prototype);
48+
obj.pointer = pointer;
49+
obj.hasReleased = false;
50+
obj.deinit = deinit;
51+
obj.registry = new FinalizationRegistry((pointer) => {
5552
deinit(pointer);
5653
});
57-
this.registry.register(this, this.pointer);
54+
obj.registry.register(this, obj.pointer);
55+
return obj;
5856
}
5957
6058
release() {
@@ -525,13 +523,11 @@ struct BridgeJSLink {
525523
var constructorLines: [String] = []
526524
constructorLines.append("static __construct(ptr) {")
527525
constructorLines.append(
528-
"return new \(klass.name)(ptr, instance.exports.bjs_\(klass.name)_deinit);".indent(count: 4)
526+
"return SwiftHeapObject.__wrap(ptr, instance.exports.bjs_\(klass.name)_deinit, \(klass.name).prototype);"
527+
.indent(count: 4)
529528
)
530529
constructorLines.append("}")
531530
constructorLines.append("")
532-
constructorLines.append("constructor(pointer, deinit) {")
533-
constructorLines.append("super(pointer, deinit);".indent(count: 4))
534-
constructorLines.append("}")
535531
jsLines.append(contentsOf: constructorLines.map { $0.indent(count: 4) })
536532

537533
if let constructor: ExportedConstructor = klass.constructor {
@@ -541,7 +537,7 @@ struct BridgeJSLink {
541537
}
542538
var funcLines: [String] = []
543539
funcLines.append("")
544-
funcLines.append("static init(\(constructor.parameters.map { $0.name }.joined(separator: ", "))) {")
540+
funcLines.append("constructor(\(constructor.parameters.map { $0.name }.joined(separator: ", "))) {")
545541
let returnExpr = thunkBuilder.callConstructor(abiName: constructor.abiName)
546542
funcLines.append(contentsOf: thunkBuilder.bodyLines.map { $0.indent(count: 4) })
547543
funcLines.append(contentsOf: thunkBuilder.cleanupLines.map { $0.indent(count: 4) })
@@ -551,7 +547,7 @@ struct BridgeJSLink {
551547
jsLines.append(contentsOf: funcLines.map { $0.indent(count: 4) })
552548

553549
dtsExportEntryLines.append(
554-
"init\(renderTSSignature(parameters: constructor.parameters, returnType: .swiftHeapObject(klass.name)));"
550+
"constructor\(renderTSSignature(parameters: constructor.parameters, returnType: .swiftHeapObject(klass.name)));"
555551
.indent(count: 4)
556552
)
557553
}

‎Plugins/BridgeJS/Tests/BridgeJSToolTests/__Snapshots__/BridgeJSLinkTests/Namespaces.Export.d.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -51,10 +51,10 @@ export interface UUID extends SwiftHeapObject {
5151
}
5252
export type Exports = {
5353
Greeter: {
54-
init(name: string): Greeter;
54+
constructor(name: string): Greeter;
5555
}
5656
Converter: {
57-
init(): Converter;
57+
constructor(): Converter;
5858
}
5959
UUID: {
6060
}

‎Plugins/BridgeJS/Tests/BridgeJSToolTests/__Snapshots__/BridgeJSLinkTests/Namespaces.Export.js

Lines changed: 13 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -60,18 +60,17 @@ export async function createInstantiator(options, swift) {
6060
const js = swift.memory.heap;
6161
/// Represents a Swift heap object like a class instance or an actor instance.
6262
class SwiftHeapObject {
63-
static __construct(ptr, deinit) {
64-
return new SwiftHeapObject(ptr, deinit);
65-
}
66-
67-
constructor(pointer, deinit) {
68-
this.pointer = pointer;
69-
this.hasReleased = false;
70-
this.deinit = deinit;
71-
this.registry = new FinalizationRegistry((pointer) => {
63+
static __wrap(pointer, deinit, prototype) {
64+
const obj = Object.create(prototype);
65+
obj.pointer = pointer;
66+
obj.hasReleased = false;
67+
obj.deinit = deinit;
68+
obj.registry = new FinalizationRegistry((pointer) => {
7269
deinit(pointer);
7370
});
7471
this.registry.register(this, this.pointer);
72+
obj.registry.register(this, obj.pointer);
73+
return obj;
7574
}
7675

7776
release() {
@@ -81,14 +80,14 @@ export async function createInstantiator(options, swift) {
8180
}
8281
class Greeter extends SwiftHeapObject {
8382
static __construct(ptr) {
84-
return new Greeter(ptr, instance.exports.bjs_Greeter_deinit);
83+
return SwiftHeapObject.__wrap(ptr, instance.exports.bjs_Greeter_deinit, Greeter.prototype);
8584
}
8685

8786
constructor(pointer, deinit) {
8887
super(pointer, deinit);
8988
}
9089

91-
static init(name) {
90+
constructor(name) {
9291
const nameBytes = textEncoder.encode(name);
9392
const nameId = swift.memory.retain(nameBytes);
9493
const ret = instance.exports.bjs_Greeter_init(nameId, nameBytes.length);
@@ -104,14 +103,14 @@ export async function createInstantiator(options, swift) {
104103
}
105104
class Converter extends SwiftHeapObject {
106105
static __construct(ptr) {
107-
return new Converter(ptr, instance.exports.bjs_Converter_deinit);
106+
return SwiftHeapObject.__wrap(ptr, instance.exports.bjs_Converter_deinit, Converter.prototype);
108107
}
109108

110109
constructor(pointer, deinit) {
111110
super(pointer, deinit);
112111
}
113112

114-
static init() {
113+
constructor() {
115114
const ret = instance.exports.bjs_Converter_init();
116115
return Converter.__construct(ret);
117116
}
@@ -124,7 +123,7 @@ export async function createInstantiator(options, swift) {
124123
}
125124
class UUID extends SwiftHeapObject {
126125
static __construct(ptr) {
127-
return new UUID(ptr, instance.exports.bjs_UUID_deinit);
126+
return SwiftHeapObject.__wrap(ptr, instance.exports.bjs_UUID_deinit, UUID.prototype);
128127
}
129128

130129
constructor(pointer, deinit) {

‎Plugins/BridgeJS/Tests/BridgeJSToolTests/__Snapshots__/BridgeJSLinkTests/SwiftClass.Export.d.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ export interface Greeter extends SwiftHeapObject {
1717
}
1818
export type Exports = {
1919
Greeter: {
20-
init(name: string): Greeter;
20+
constructor(name: string): Greeter;
2121
}
2222
takeGreeter(greeter: Greeter): void;
2323
}

‎Plugins/BridgeJS/Tests/BridgeJSToolTests/__Snapshots__/BridgeJSLinkTests/SwiftClass.Export.js

Lines changed: 10 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -60,18 +60,17 @@ export async function createInstantiator(options, swift) {
6060
const js = swift.memory.heap;
6161
/// Represents a Swift heap object like a class instance or an actor instance.
6262
class SwiftHeapObject {
63-
static __construct(ptr, deinit) {
64-
return new SwiftHeapObject(ptr, deinit);
65-
}
66-
67-
constructor(pointer, deinit) {
68-
this.pointer = pointer;
69-
this.hasReleased = false;
70-
this.deinit = deinit;
71-
this.registry = new FinalizationRegistry((pointer) => {
63+
static __wrap(pointer, deinit, prototype) {
64+
const obj = Object.create(prototype);
65+
obj.pointer = pointer;
66+
obj.hasReleased = false;
67+
obj.deinit = deinit;
68+
obj.registry = new FinalizationRegistry((pointer) => {
7269
deinit(pointer);
7370
});
7471
this.registry.register(this, this.pointer);
72+
obj.registry.register(this, obj.pointer);
73+
return obj;
7574
}
7675

7776
release() {
@@ -81,14 +80,14 @@ export async function createInstantiator(options, swift) {
8180
}
8281
class Greeter extends SwiftHeapObject {
8382
static __construct(ptr) {
84-
return new Greeter(ptr, instance.exports.bjs_Greeter_deinit);
83+
return SwiftHeapObject.__wrap(ptr, instance.exports.bjs_Greeter_deinit, Greeter.prototype);
8584
}
8685

8786
constructor(pointer, deinit) {
8887
super(pointer, deinit);
8988
}
9089

91-
static init(name) {
90+
constructor(name) {
9291
const nameBytes = textEncoder.encode(name);
9392
const nameId = swift.memory.retain(nameBytes);
9493
const ret = instance.exports.bjs_Greeter_init(nameId, nameBytes.length);

0 commit comments

Comments
 (0)