Skip to content

Commit ce37688

Browse files
RafaelGSSruyadorno
authored andcommitted
src: remove unnecessary todo
PR-URL: #49227 Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com> Reviewed-By: James M Snell <jasnell@gmail.com>
1 parent a5dd057 commit ce37688

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
@@ -2116,7 +2116,6 @@ static inline Maybe<void> CheckOpenPermissions(Environment* env,
21162116
const int write_as_side_effect = flags & (UV_FS_O_APPEND | UV_FS_O_CREAT |
21172117
UV_FS_O_TRUNC | UV_FS_O_TEMPORARY);
21182118

2119-
// TODO(rafaelgss): it can be optimized to avoid two permission checks
21202119
auto pathView = path.ToStringView();
21212120
if (rwflags != UV_FS_O_WRONLY) {
21222121
THROW_IF_INSUFFICIENT_PERMISSIONS(

0 commit comments

Comments
 (0)