Skip to content

Commit f7879b7

Browse files
committed
[test] Remove EMSCRIPTEN_KEEPALIVE from _ReportResult. NFC
On the emscripten side we have `reportResultToServer` already.
1 parent 82182e6 commit f7879b7

File tree

3 files changed

+4
-6
lines changed

3 files changed

+4
-6
lines changed

test/pthread/call_async_on_main_thread.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,6 @@ addToLibrary({
77
console.error('This function should be getting called on the main thread!');
88
}
99
console.log('got ' + param1 + ' ' + param2 + ' ' + param3);
10-
__ReportResult(param1 + param2 * param3);
10+
reportResultToServer(param1 + param2 * param3);
1111
}
1212
});

test/report_result.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,11 +21,11 @@ extern "C" {
2121

2222
#if defined __EMSCRIPTEN__ && !defined EMTEST_NODE
2323

24-
void EMSCRIPTEN_KEEPALIVE _ReportResult(int result) {
24+
void _ReportResult(int result) {
2525
EM_ASM(reportResultToServer($0), result);
2626
}
2727

28-
void EMSCRIPTEN_KEEPALIVE _MaybeReportResult(int result) {
28+
void _MaybeReportResult(int result) {
2929
EM_ASM(maybeReportResultToServer($0), result);
3030
}
3131

test/webaudio/create_webaudio.c

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -23,9 +23,7 @@ int main()
2323
startButton.onclick = () => {
2424
if (audioContext.state != 'running') {
2525
audioContext.resume();
26-
#ifdef REPORT_RESULT
27-
__ReportResult(0);
28-
#endif
26+
reportResultToServer(0);
2927
} else {
3028
audioContext.suspend();
3129
}

0 commit comments

Comments
 (0)