Skip to content

Add a terminated dependency and use NulTerminatedStr for inputs#113

Closed
madsmtm wants to merge 1 commit into
masterfrom
nullterminated-strings
Closed

Add a terminated dependency and use NulTerminatedStr for inputs#113
madsmtm wants to merge 1 commit into
masterfrom
nullterminated-strings

Conversation

@madsmtm
Copy link
Copy Markdown
Owner

@madsmtm madsmtm commented Jan 6, 2022

@SSheldon's original work on SSheldon/rust-objc#54.

@madsmtm madsmtm added the enhancement New feature or request label Jan 6, 2022
@madsmtm madsmtm mentioned this pull request Jan 6, 2022
80 tasks
@madsmtm madsmtm added wontfix and removed wontfix labels Jun 24, 2022
@madsmtm
Copy link
Copy Markdown
Owner Author

madsmtm commented Sep 5, 2023

I don't think this makes sense to do, since actually, none of these APIs enforce valid UTF-8 strings.

Rather, we should just expose the inherent CStr-nature of these APIs some more, see #564.

@madsmtm madsmtm closed this Sep 5, 2023
@madsmtm madsmtm deleted the nullterminated-strings branch September 5, 2023 20:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants