Skip to content

Conversation

@ctrlaltmilk
Copy link
Contributor

Fairly simple, just got annoyed while tinkering with a personal project and thought it might be nice to contribute something to help. Both macros take a string literal and convert it to the respective CStr literal.

@GabrielMajeri
Copy link
Collaborator

Thanks for the PR! It looks good overall. Could you also add an entry to the CHANGELOG.md file?

@ctrlaltmilk
Copy link
Contributor Author

Got it, will do!

@nicholasbishop
Copy link
Member

Thanks for the PR! I think this is a very nice ease-of-use improvement to strings.

@ctrlaltmilk
Copy link
Contributor Author

Alright, that should add both the doc comments and allow for the full UTF-16 range. Thanks for catching that by the way!

@nicholasbishop
Copy link
Member

There are a few clippy lints that need resolving, otherwise lgtm!

@GabrielMajeri GabrielMajeri changed the title Create macros for making CStr16 and CStr8 literals Create macros for making CStr16 and CStr8 literals May 16, 2022
@GabrielMajeri GabrielMajeri merged commit 805e744 into rust-osdev:main May 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants