Skip to content
This repository has been archived by the owner on Mar 7, 2021. It is now read-only.

Refactor the CStr API to reduce the need for nightly features #271

Merged
merged 1 commit into from
Sep 6, 2020

Conversation

alex
Copy link
Member

@alex alex commented Sep 5, 2020

No description provided.

@alex alex force-pushed the refactor-cstr-to-reduce-nightly branch 3 times, most recently from 697fd82 to 2fcb49f Compare September 5, 2020 04:23
@alex alex requested a review from geofft September 5, 2020 04:30
@alex alex force-pushed the refactor-cstr-to-reduce-nightly branch from 2fcb49f to 6979f48 Compare September 5, 2020 13:43
@geofft
Copy link
Collaborator

geofft commented Sep 6, 2020

Hrrrmm. I'm fine with this, but I think we should plan to revert it once we can, because now our API looks pretty different from str and OsStr. That is, I think we should leave const_str_as_bytes (or some other way of accomplishing the same goal) on our list of things we need from Rust upstream.

I don't think const_str_as_bytes is gonna be hard to stabilize (though maybe we should check?), so my personal feeling is that it would be fine to rely on the feature for now - we're on nightly for a while because of alloc, anyway.

Approving, I'll leave it up to you as to whether you want to merge it for now.

@alex
Copy link
Member Author

alex commented Sep 6, 2020

Just to be clear, the feature in question here is const_raw_ptr_deref, not const_str_as_bytes.

@alex alex merged commit abc9791 into master Sep 6, 2020
@alex alex deleted the refactor-cstr-to-reduce-nightly branch September 6, 2020 22:39
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