Skip to content

Conversation

liamzebedee
Copy link

No description provided.

Copy link
Owner

@andrewmilson andrewmilson left a comment

Choose a reason for hiding this comment

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

TY. Can scripts/ and proof.* be removed please? I'd prefer to only have things that go hand in hand with the library.

}

pub struct RegisterStates(Vec<RegisterState>);
pub struct RegisterStates(pub Vec<RegisterState>);
Copy link
Owner

Choose a reason for hiding this comment

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

The types already implement deref. Is there a need to make this public?

Copy link
Author

Choose a reason for hiding this comment

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

Again I should've said WIP. Short-term fix to get this working:

https://github.com/ponderingdemocritus/darude/blob/main/wasm/src/lib.rs#L172-L183

Copy link
Author

Choose a reason for hiding this comment

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

IDK how to use deref to solve it, so just publicised it for now (we don't have to merge it in).

src/main.rs Outdated
Comment on lines -23 to -26
#[cfg(not(target_env = "msvc"))]
#[global_allocator]
static GLOBAL: Jemalloc = Jemalloc;

Copy link
Owner

Choose a reason for hiding this comment

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

👍

@liamzebedee
Copy link
Author

@andrewmilson sorry forgot to mark as WIP

@liamzebedee liamzebedee changed the title Darude mods [wip] Darude mods Jan 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants