diff options
author | Asher Gomez <ashersaupingomez@gmail.com> | 2024-01-24 11:07:06 +1100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-01-24 01:07:06 +0100 |
commit | 47620641e7455a0f9df82d17ad0405693e4427a4 (patch) | |
tree | 578a8c3fa25c157fb7cfa226d0f4b66b54995cda /ext/fs/30_fs.js | |
parent | 2f47ec6c3a583c8323a06c386feeaee4fbf75edc (diff) |
feat: `FsFile.sync()` and `FsFile.syncSync()` (#22017)
This change:
1. Implements `Deno.FsFile.sync()` and `Deno.FsFile.syncSync()`.
2. Deprecates `Deno.fsync()` and `Deno.fsyncSync()` for removal in Deno
v2, in favour of the above corresponding methods.
Related #21995
---------
Co-authored-by: Bartek IwaĆczuk <biwanczuk@gmail.com>
Diffstat (limited to 'ext/fs/30_fs.js')
-rw-r--r-- | ext/fs/30_fs.js | 18 |
1 files changed, 18 insertions, 0 deletions
diff --git a/ext/fs/30_fs.js b/ext/fs/30_fs.js index 3c888f1f1..6923741ee 100644 --- a/ext/fs/30_fs.js +++ b/ext/fs/30_fs.js @@ -576,10 +576,20 @@ async function fdatasync(rid) { } function fsyncSync(rid) { + internals.warnOnDeprecatedApi( + "Deno.fsyncSync()", + new Error().stack, + "Use `file.syncSync()` instead.", + ); op_fs_fsync_sync(rid); } async function fsync(rid) { + internals.warnOnDeprecatedApi( + "Deno.fsync()", + new Error().stack, + "Use `file.sync()` instead.", + ); await op_fs_fsync_async(rid); } @@ -739,6 +749,14 @@ class FsFile { return this.#writable; } + async sync() { + await op_fs_fsync_async(this.rid); + } + + syncSync() { + op_fs_fsync_sync(this.rid); + } + [SymbolDispose]() { core.tryClose(this.rid); } |