Skip to content

Commit

Permalink
fix(wasi): return flags from fd_fdstat_get (denoland/deno#7112)
Browse files Browse the repository at this point in the history
  • Loading branch information
caspervonb authored and denobot committed Feb 1, 2021
1 parent 7fab5a8 commit 8ec727d
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 3 deletions.
8 changes: 6 additions & 2 deletions wasi/snapshot_preview1.ts
Original file line number Diff line number Diff line change
Expand Up @@ -307,14 +307,17 @@ export default class Context {

this.fds = [
{
fdflags: FDFLAGS_APPEND,
type: FILETYPE_CHARACTER_DEVICE,
handle: Deno.stdin,
},
{
fdflags: FDFLAGS_APPEND,
type: FILETYPE_CHARACTER_DEVICE,
handle: Deno.stdout,
},
{
fdflags: FDFLAGS_APPEND,
type: FILETYPE_CHARACTER_DEVICE,
handle: Deno.stderr,
},
Expand Down Expand Up @@ -521,7 +524,7 @@ export default class Context {

const view = new DataView(this.memory.buffer);
view.setUint8(stat_out, entry.type);
view.setUint16(stat_out + 4, 0, true); // TODO
view.setUint16(stat_out + 2, entry.fdflags, true);
view.setBigUint64(stat_out + 8, 0n, true); // TODO
view.setBigUint64(stat_out + 16, 0n, true); // TODO

Expand Down Expand Up @@ -1155,9 +1158,9 @@ export default class Context {
// directory this way so there's no native fstat but Deno.open
// doesn't work with directories on windows so we'll have to work
// around it for now.

const entries = Array.from(Deno.readDirSync(path));
const opened_fd = this.fds.push({
fdflags,
entries,
path,
}) - 1;
Expand Down Expand Up @@ -1237,6 +1240,7 @@ export default class Context {

const handle = Deno.openSync(path, options);
const opened_fd = this.fds.push({
fdflags,
handle,
path,
}) - 1;
Expand Down
2 changes: 1 addition & 1 deletion wasi/testdata
Submodule testdata updated from 419874 to bbdc8e

0 comments on commit 8ec727d

Please sign in to comment.