From 9d19439e40683dddb4f896e9f5c7fc605899d1d4 Mon Sep 17 00:00:00 2001
From: Rafael Gonzaga <rafael.nunu@hotmail.com>
Date: Thu, 21 Sep 2023 18:55:01 -0300
Subject: [PATCH] src: remove unnecessary todo

PR-URL: https://github.com/nodejs/node/pull/49227
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
---
 src/node_file.cc | 1 -
 1 file changed, 1 deletion(-)

diff --git a/src/node_file.cc b/src/node_file.cc
index 1e2d1c85322dab..a19c4fa5bb7893 100644
--- a/src/node_file.cc
+++ b/src/node_file.cc
@@ -2081,7 +2081,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(