xmppftw/xmpp-rs
Watch
1
0
Fork
You've already forked xmpp-rs
 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

minidom: Don't borrow prefix in Prefixes.get

Browse source 
Signed-off-by: Maxime “pep” Buquet <pep@bouah.net>
This commit is contained in:
Maxime “pep” Buquet 2020-03-29 21:25:18 +02:00
commit 770dff7cb0
2 changed files with 3 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

2
minidom-rs/src/element.rs
View file

@ -1093,7 +1093,7 @@ mod tests {
assert_eq!(elem.name(), String::from("bar")); assert_eq!(elem.name(), String::from("bar"));
assert_eq!(elem.ns(), String::from("ns1")); assert_eq!(elem.ns(), String::from("ns1"));
// Ensure the prefix is properly added to the store // Ensure the prefix is properly added to the store
assert_eq!(elem.prefixes.get(Some(String::from("foo"))), Some(&String::from("ns1"))); assert_eq!(elem.prefixes.get(&Some(String::from("foo"))), Some(&String::from("ns1")));
} }
#[test] #[test]

4
minidom-rs/src/prefixes.rs
View file

@ -49,8 +49,8 @@ impl Prefixes {
&self.prefixes &self.prefixes
} }
pub fn get(&self, prefix: Prefix) -> Option<&Namespace> { pub fn get(&self, prefix: &Prefix) -> Option<&Namespace> {
self.prefixes.get(&prefix) self.prefixes.get(prefix)
} }
pub(crate) fn insert<S: Into<Namespace>>(&mut self, prefix: Prefix, namespace: S) { pub(crate) fn insert<S: Into<Namespace>>(&mut self, prefix: Prefix, namespace: S) {