perf(core): remove wasteful clones

Author: JamieMason

src/commands/ui/instance.rs

@@ -11,7 +11,8 @@ use {
 };
 
 pub fn print(ctx: &Context, instance: &Instance, group_variant: &VersionGroupVariant) {
-  match &instance.state.borrow().clone() {
+  let state = instance.state.borrow();
+  match &*state {
     InstanceState::Valid(ValidInstance::IsIgnored) => print_ignored(ctx, instance, group_variant),
     InstanceState::Valid(_) => print_valid(ctx, instance, group_variant),
     InstanceState::Invalid(InvalidInstance::Unfixable(_)) => print_unfixable(ctx, instance, group_variant),
@@ -145,7 +146,7 @@ pub fn get_location(ctx: &Context, instance: &Instance) -> String {
 }
 
 fn get_state_name(instance: &Instance, group_variant: &VersionGroupVariant) -> String {
-  let state = instance.state.borrow().clone();
+  let state = instance.state.borrow();
   let state_name = state.get_name();
   // Issues related to whether a specifier is the highest or lowest semver are
   // all the same logic internally, so we have combined enum branches for