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

Compile against Rust Stable #17

Closed
3 of 5 tasks
kkawakam opened this issue Dec 20, 2015 · 10 comments
Closed
3 of 5 tasks

Compile against Rust Stable #17

kkawakam opened this issue Dec 20, 2015 · 10 comments

Comments

@kkawakam
Copy link
Owner

kkawakam commented Dec 20, 2015

Currently we take advantage of some non-stabilized rust APIs but eventually we will want to be able to compile against Rust stable. The below links are tracking issues that need to be closed out before we can compile against rust stable.

Update June 6 2016: #38 introduced workaround changes that allowed rustyline to be compiled against rust stable (1.9.0). These changes should eventually replaced by Read::chars and encode_utf8 after they become stabilized in the standard library.

TO-DO

@pwrdwnsys
Copy link

Is this now complete with the changes merged from #38 ?

@kkawakam
Copy link
Owner Author

kkawakam commented Jun 7, 2016

@pastoraleman I'm going to leave this open because we introduced some work-arounds in #38 to compile against 1.9.0 stable. When Read::chars and encode_utf8 become stabilized, they should replace the work-arounds introduced in #38.

I'll update the issue to reflect that change

@Yamakaky
Copy link

Yamakaky commented Nov 4, 2016

Did you consider using https://crates.io/crates/unicode-segmentation?

@gwenn
Copy link
Collaborator

gwenn commented Nov 4, 2016

@Yamakaky Correct me if I'm wrong, but it seems to me that the only places where this crate can be used is in Completer and "word" actions where the boundaries are specific (not conforming to unicode standard). And we still need to iterate on chars when reading the standard input stream...

@Yamakaky
Copy link

Yamakaky commented Nov 4, 2016

Oh, ok.
Opened unicode-rs/unicode-segmentation#7

@jonas-schievink
Copy link

Looks like this can now be closed! :)

@gwenn
Copy link
Collaborator

gwenn commented May 7, 2017

Negative !
Read::chars is still unstable.
But, encode_unicode can be removed (encode_utf8 is stabilised).

@jonas-schievink
Copy link

Oh, I see now. It compiles against stable, but there's a few workarounds/reimplementations for features that aren't yet stable.

@gwenn
Copy link
Collaborator

gwenn commented May 1, 2018

Read::chars has not been stabilized but removed !

@gwenn
Copy link
Collaborator

gwenn commented Aug 26, 2018

2.0 released

@gwenn gwenn closed this as completed Aug 26, 2018
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

No branches or pull requests

5 participants