Skip to content

Commit

Permalink
Fix for none proxying mode (emscripten-core#22824)
Browse files Browse the repository at this point in the history
It looks like this mode was broken from the moment it as added back in
PR emscripten-core#20110. However it went unnoticed since it was only being used with a
single function: `__syscall_fadvise64`.

Split out from emscripten-core#22820.
  • Loading branch information
sbc100 authored Oct 31, 2024
1 parent d097f6d commit 753b3bf
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/jsifier.mjs
Original file line number Diff line number Diff line change
Expand Up @@ -326,7 +326,7 @@ function(${args}) {
if (proxyingMode !== 'sync' && proxyingMode !== 'async' && proxyingMode !== 'none') {
throw new Error(`Invalid proxyingMode ${symbol}__proxy: '${proxyingMode}' specified!`);
}
if (SHARED_MEMORY) {
if (SHARED_MEMORY && proxyingMode != 'none') {
const sync = proxyingMode === 'sync';
if (PTHREADS) {
snippet = modifyJSFunction(snippet, (args, body, async_, oneliner) => {
Expand Down

0 comments on commit 753b3bf

Please sign in to comment.