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.

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

extensions/amd: Remove unnecessary CString allocations in fn load #533

Merged
merged 2 commits into from
Dec 28, 2021
Merged

extensions/amd: Remove unnecessary CString allocations in fn load #533

merged 2 commits into from
Dec 28, 2021

Commits on Dec 27, 2021

  1. extensions/amd: Remove unnecessary CString allocations in fn load 

    Following the changes in a053c6a ("Remove unnecessary CString allocation
when loading functions (#379)") this addresses the remainder of string
allocations in manual extension loading code.
    @MarijnS95
    MarijnS95 committed Dec 27, 2021
    Configuration menu
    Copy the full SHA
    0484777 View commit details
    Browse the repository at this point in the history

  2. examples: Remove remaining CString allocations on string literals 

    These allocations and runtime assertions can all be replaced with a
`CStr` directly wrapping a binary-string literal, as long as it is
null-terminated.
    @MarijnS95
    MarijnS95 committed Dec 27, 2021
    Configuration menu
    Copy the full SHA
    b98d619 View commit details
    Browse the repository at this point in the history