Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Close the correct pipelines when evicted from navigation context. #966

Merged
merged 4 commits into from Sep 24, 2013
Merged
Changes from 1 commit
Commits
File filter...
Filter file types
Jump to…
Jump to file
Failed to load files.

Always

Just for now

Next

exit the correct evicted pipelines

  • Loading branch information
Tim Kuehn
Tim Kuehn committed Sep 24, 2013
commit d77a07388f9acd1d45533d37544b09de98e4f36d
@@ -770,12 +770,13 @@ impl Constellation {
match navigation_type {
constellation_msg::Load => {
let evicted = self.navigation_context.load(frame_tree);
let mut exited = HashSet::new();
// exit any pipelines that don't exist outside the evicted frame trees
for frame_tree in evicted.iter() {
// exit any pipelines that don't exist outside the evicted frame trees
for frame in frame_tree.iter() {
if !self.navigation_context.contains(frame.pipeline.id) {
frame_tree.pipeline.exit();
self.pipelines.remove(&frame_tree.pipeline.id);
for @FrameTree { pipeline, _ } in frame_tree.iter() {
if !self.navigation_context.contains(pipeline.id) {
pipeline.exit();
self.pipelines.remove(&pipeline.id);
}
}
}
ProTip! Use n and p to navigate between commits in a pull request.
You can’t perform that action at this time.