From ce37688bac6f488f2b59482c14ff6f2629396b33 Mon Sep 17 00:00:00 2001 From: Rafael Gonzaga 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 Reviewed-By: James M Snell --- src/node_file.cc | 1 - 1 file changed, 1 deletion(-) diff --git a/src/node_file.cc b/src/node_file.cc index 9040e00eb0fdac..39d8ac078b435b 100644 --- a/src/node_file.cc +++ b/src/node_file.cc @@ -2116,7 +2116,6 @@ static inline Maybe 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(