Skip to content

Commit

Permalink
trace keys as well
Browse files Browse the repository at this point in the history
  • Loading branch information
ajnirp committed Sep 28, 2014
1 parent a2220f1 commit e50e704
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion components/script/dom/bindings/trace.rs
Expand Up @@ -211,10 +211,11 @@ impl<T: JSTraceable> JSTraceable for Option<T> {
}
}

impl<K: Eq+Hash, V: JSTraceable> JSTraceable for HashMap<K, V> {
impl<K: Eq+Hash+JSTraceable, V: JSTraceable> JSTraceable for HashMap<K, V> {
#[inline]
fn trace(&self, trc: *mut JSTracer) {
for e in self.iter() {
e.val0().trace(trc);
e.val1().trace(trc);
}
}
Expand Down

5 comments on commit e50e704

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

saw approval from Manishearth
at ajnirp@e50e704

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merging wenderen/servo/JSTraceable.key = e50e704 into bors-servo-integration-3508-JSTraceable.key

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wenderen/servo/JSTraceable.key = e50e704 merged ok, testing candidate = 4220567

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fast-forwarding master to bors-servo-integration-3508-JSTraceable.key = 4220567

Please sign in to comment.