Accept Option<&str> instead of &Option<String>, etc (Qiskit#12593)

* Accept Option<&str> instead of &Option<String>, etc

In a few places, this removes unnecessary object copies.

Accept a wider range of input types, and more idiomatic input types
in a few functions. This affects code added in the gates-in-rust PR.

The great majority of the changes here were obsoleted by Qiskit#12594.
The original commit has been cherry picked on top of main.

* Remove unnecessary as_ref()

crates/circuit/src/circuit_data.rs

@@ -1048,13 +1048,7 @@ impl CircuitData {
         Ok(PySet::new_bound(py, self.param_table.uuid_map.values())?.unbind())
     }
 
-    pub fn pop_param(
-        &mut self,
-        py: Python,
-        uuid: u128,
-        name: String,
-        default: PyObject,
-    ) -> PyObject {
+    pub fn pop_param(&mut self, py: Python, uuid: u128, name: &str, default: PyObject) -> PyObject {
         match self.param_table.pop(uuid, name) {
             Some(res) => res.into_py(py),
             None => default.clone_ref(py),

crates/circuit/src/parameter_table.rs

@@ -153,8 +153,8 @@ impl ParamTable {
         self.uuid_map.clear();
     }
 
-    pub fn pop(&mut self, key: u128, name: String) -> Option<ParamEntry> {
-        self.names.remove(&name);
+    pub fn pop(&mut self, key: u128, name: &str) -> Option<ParamEntry> {
+        self.names.remove(name);
         self.uuid_map.remove(&key);
         self.table.remove(&key)
     }