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

Use c-string literals and update trampolines #2590

Merged
merged 2 commits into from
Mar 21, 2024

Conversation

konstin
Copy link
Member

@konstin konstin commented Mar 21, 2024

Rust 1.77 has stabilized c-string literals (c"<string>"): https://doc.rust-lang.org/nightly/edition-guide/rust-2021/c-string-literals.html. This PR replaces the usages of the custom c-string literal macro in the trampoline with the new syntax.

Rust 1.77 has stabilized c-string literals (`c"<string>"`): https://doc.rust-lang.org/nightly/edition-guide/rust-2021/c-string-literals.html. This PR replaces the usages of the custom c-string literal macro in the trampoline with the new syntax.
@konstin konstin added the enhancement New feature or request label Mar 21, 2024
@konstin konstin enabled auto-merge (squash) March 21, 2024 15:18
Copy link
Member

@zanieb zanieb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sweet

@zanieb
Copy link
Member

zanieb commented Mar 21, 2024

Is this external facing? Or internal?

@konstin
Copy link
Member Author

konstin commented Mar 21, 2024

It's internal but i've updated the trampolines so i wanted to put it into the release notes

@konstin konstin merged commit 2375008 into main Mar 21, 2024
31 checks passed
@konstin konstin deleted the konsti/use-c-string-literals branch March 21, 2024 15:36
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