Skip to content

Fix building under current Rust master (std::vec dep).#5

Open
tekacs wants to merge 2 commits intoerickt:masterfrom
tekacs:master
Open

Fix building under current Rust master (std::vec dep).#5
tekacs wants to merge 2 commits intoerickt:masterfrom
tekacs:master

Conversation

@tekacs
Copy link

@tekacs tekacs commented Oct 4, 2014

Changes are just:

use std::vec;
...
Vec -> vec::Vec

Do switch this out if there's a tidier way of doing this that I've ignored. The persistent overlap between Vec/Vec is the source of the verbosity.

Hopefully a viable approach?
@tekacs
Copy link
Author

tekacs commented Oct 4, 2014

... aaand the tests also needed re-writing. Fixing.

@tekacs
Copy link
Author

tekacs commented Oct 4, 2014

And now the last fix I have to leave to someone who knows what they're doing in this context, whilst I go and bring my Rust knowledge up to the current language version. :)

Presumably the trait for as_slice (via std::str::Str) needs to be brought into the current scope without overlapping with super::Str.

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.

1 participant