diff options
| author | David Sherret <dsherret@users.noreply.github.com> | 2023-04-24 21:07:48 -0400 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2023-04-24 21:07:48 -0400 |
| commit | 5b4a9b48aeabded174cc161bfdafb6529cffbca1 (patch) | |
| tree | 77a2e82cfe04e2c1c04200496499a11f6d53b7e9 /cli/tsc | |
| parent | 667acb075c0f1fb62bc1177f46b550e9dd506c00 (diff) | |
refactor(ext/node): enforce interior mutable for `NodePermissions` to remove clones (#18831)
We can make `NodePermissions` rely on interior mutability (which the
`PermissionsContainer` is already doing) in order to not have to clone
everything all the time. This also reduces the chance of an accidental
`borrow` while `borrrow_mut`.
Diffstat (limited to 'cli/tsc')
| -rw-r--r-- | cli/tsc/mod.rs | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/cli/tsc/mod.rs b/cli/tsc/mod.rs index aa589a1ca..b77f39fd5 100644 --- a/cli/tsc/mod.rs +++ b/cli/tsc/mod.rs @@ -639,7 +639,7 @@ fn resolve_graph_specifier_types( let maybe_resolution = node_resolver.resolve_npm_reference( &module.nv_reference, NodeResolutionMode::Types, - &mut PermissionsContainer::allow_all(), + &PermissionsContainer::allow_all(), )?; Ok(Some(NodeResolution::into_specifier_and_media_type( maybe_resolution, @@ -679,7 +679,7 @@ fn resolve_non_graph_specifier_types( specifier, referrer, NodeResolutionMode::Types, - &mut PermissionsContainer::allow_all(), + &PermissionsContainer::allow_all(), ) .ok() .flatten(), @@ -692,7 +692,7 @@ fn resolve_non_graph_specifier_types( let maybe_resolution = node_resolver.resolve_npm_req_reference( &npm_ref, NodeResolutionMode::Types, - &mut PermissionsContainer::allow_all(), + &PermissionsContainer::allow_all(), )?; Ok(Some(NodeResolution::into_specifier_and_media_type( maybe_resolution, |