Skip to content

Commit 85172c3

Browse files
authored
src: remove unnecessary todo
PR-URL: #49227 Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: James M Snell <[email protected]>
1 parent 3116c37 commit 85172c3

File tree

1 file changed

+0
-1
lines changed

1 file changed

+0
-1
lines changed

src/node_file.cc

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2081,7 +2081,6 @@ static inline Maybe<void> CheckOpenPermissions(Environment* env,
20812081
const int write_as_side_effect = flags & (UV_FS_O_APPEND | UV_FS_O_CREAT |
20822082
UV_FS_O_TRUNC | UV_FS_O_TEMPORARY);
20832083

2084-
// TODO(rafaelgss): it can be optimized to avoid two permission checks
20852084
auto pathView = path.ToStringView();
20862085
if (rwflags != UV_FS_O_WRONLY) {
20872086
THROW_IF_INSUFFICIENT_PERMISSIONS(

0 commit comments

Comments
 (0)