Skip to content

Fix compilation for rust nightly#154

Merged
bluss merged 1 commit intopetgraph:masterfrom
daboross:fix-2017-06-14-nightly
Jun 16, 2017
Merged

Fix compilation for rust nightly#154
bluss merged 1 commit intopetgraph:masterfrom
daboross:fix-2017-06-14-nightly

Conversation

@daboross
Copy link
Contributor

Very small change to remove ambiguity between Ord::max and IndexType::max in two locations.

I don't directly use this library, only as an indirect dependency, but I hope this is helpful.

Fixes #153.

Fixes by removing ambiguity between Ord::max and IndexType::max in two locations.
@nikomatsakis
Copy link

(This also affects some older branches, fyi.)

@daboross
Copy link
Contributor Author

True, thanks for mentioning that. I guess I'll submit another PR towards branch-0.2? I'm not sure there's a branch already existing for 0.3.*.

@nikomatsakis
Copy link

@daboross I don't know how @bluss manages it.

@bluss bluss merged commit f0f6e39 into petgraph:master Jun 16, 2017
@bluss
Copy link
Member

bluss commented Jun 16, 2017

Thank you

@bluss
Copy link
Member

bluss commented Jun 16, 2017

Fixed in 0.4.5

@daboross daboross deleted the fix-2017-06-14-nightly branch June 16, 2017 19:57
bluss pushed a commit that referenced this pull request Jul 27, 2017
Fixes by removing ambiguity between Ord::max and IndexType::max in two locations.
illicitonion added a commit to illicitonion/pants that referenced this pull request Sep 1, 2017
rust-lang/rust#42496 means that 0.4.4 no longer
compiles with the most recently released rust toolchain. This was fixed
in petgraph/petgraph#154 which is in 0.4.5.
khuey pushed a commit to khuey/petgraph that referenced this pull request Feb 26, 2018
Fixes by removing ambiguity between Ord::max and IndexType::max in two locations.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants