Restructure Paths::path_response

[larseggert]

I think that you can restructure this little piece of code to avoid the extra break.

let Some(primary) = self.migration_target.as_mut() else { break; }
if Rc::ptr_eq(target, primary) {
  let p = primary.take();
  mem::drop(self.select_primary(&p, now));
  return true;
}
break;

Once we're on 1.80 MSRV, we can use Option::take_if and simplify further.

if let Some(primary) = self.migration_target.take_if(|p| Rc::ptr_eq(target, p)) 
  mem::drop(self.select_primary(&primary, now));
  return true;
}
break;

Originally posted by @martinthomson in #2294 (comment)