Skip to content
This repository has been archived by the owner on May 11, 2020. It is now read-only.

Update to run on rust nightly 1.15.0 #23

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

acmiyaguchi
Copy link

llamadb is broken in the current rust nightly 1.15.0 release.

The first change is using the Cow:: smart pointer constructor instead of into_cow, since it has been depreciated. The other change is to the time::Duration module, where span was depreciated. The idiomatic way to time a function is to use Instant::now() with the corresponding elapsed() function.

@stalkerg
Copy link

stalkerg commented Dec 21, 2016

It can be working on the latest stable Rust release?

@acmiyaguchi
Copy link
Author

@stalkerg I took a look at it and I'm not sure if I want to mess around with it. Associated types are behind a feature gate and are being used in the parser for generic programming. I'd rather spend my time doing some other things (namely adding tests).

For more reference, here's the tracking issue for associated types if you're curious on how they work and want to tackle it yourself.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants