src: remove unnecessary todo

PR-URL: #49227 Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com> Reviewed-By: James M Snell <jasnell@gmail.com>
nodejs · Sep 28, 2023 · ce37688 · ce37688
1 parent a5dd057
commit ce37688
Showing 1 changed file with 0 additions and 1 deletion.
diff --git a/src/node_file.cc b/src/node_file.cc
@@ -2116,7 +2116,6 @@ static inline Maybe<void> CheckOpenPermissions(Environment* env,
   const int write_as_side_effect = flags & (UV_FS_O_APPEND | UV_FS_O_CREAT |
                                             UV_FS_O_TRUNC | UV_FS_O_TEMPORARY);
 
-  // TODO(rafaelgss): it can be optimized to avoid two permission checks
   auto pathView = path.ToStringView();
   if (rwflags != UV_FS_O_WRONLY) {
     THROW_IF_INSUFFICIENT_PERMISSIONS(