Skip to content

Commit

Permalink
Sort trait bounds.
Browse files Browse the repository at this point in the history
Closes #6334
  • Loading branch information
jfager committed Jul 25, 2014
1 parent d30776e commit 08f033b
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
3 changes: 2 additions & 1 deletion src/librustc/middle/typeck/collect.rs
Expand Up @@ -1218,6 +1218,8 @@ fn ty_generics(ccx: &CrateCtxt,

check_bounds_compatible(ccx.tcx, &param_bounds, ident, span);

param_bounds.trait_bounds.sort_by(|a,b| a.def_id.cmp(&b.def_id));

param_bounds
}

Expand Down Expand Up @@ -1340,4 +1342,3 @@ fn check_method_self_type<RS:RegionScope>(
_ => {}
}
}

2 changes: 0 additions & 2 deletions src/test/run-pass/issue-6334.rs
Expand Up @@ -8,8 +8,6 @@
// option. This file may not be copied, modified, or distributed
// except according to those terms.

// ignore-test

// Tests that everything still compiles and runs fine even when
// we reorder the bounds.

Expand Down

5 comments on commit 08f033b

@bors
Copy link
Contributor

@bors bors commented on 08f033b Aug 6, 2014

Choose a reason for hiding this comment

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

saw approval from pnkfelix
at jfager@08f033b

@bors
Copy link
Contributor

@bors bors commented on 08f033b Aug 6, 2014

Choose a reason for hiding this comment

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

merging jfager/rust/r6334 = 08f033b into auto

@bors
Copy link
Contributor

@bors bors commented on 08f033b Aug 6, 2014

Choose a reason for hiding this comment

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

jfager/rust/r6334 = 08f033b merged ok, testing candidate = dd20f09

@bors
Copy link
Contributor

@bors bors commented on 08f033b Aug 6, 2014

Choose a reason for hiding this comment

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

fast-forwarding master to auto = dd20f09

Please sign in to comment.