Update book docs to use Ruby::str_new #554

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.

Sign up for GitHub

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

Jump to bottom

Open

ryanseys wants to merge 1 commit into oxidize-rb:main from ryanseys:patch-1

ryanseys commented May 28, 2025

Update book docs to use new magnus Ruby::str_new instead of RString::new (which is now on old-api magnus feature)

Also shortened from "Hello Ruby!" to just "Ruby!" and reformatted alignment of comments to save some space on screen.

OLD source: https://github.com/matsadler/magnus/blob/b83762a95462fa87a1fae445f243e7eb9ae5c8ab/src/r_string.rs#L313-L340

NEW source: https://github.com/matsadler/magnus/blob/b83762a95462fa87a1fae445f243e7eb9ae5c8ab/src/r_string.rs#L46-L69


          Update book to use Ruby::str_new

8b8a441

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet