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

remove weealloc suggestion or add warning about OOM in wasm section #129

Closed
TomPridham opened this issue Mar 6, 2022 · 1 comment
Closed

Comments

@TomPridham
Copy link

it seems like weealloc doesn't work terribly well with bevy. it grows memory at a fairly quick pace and will cause an OOM with just the basic program below.
i'm also not the only person to encounter issues with it: rustwasm/wee_alloc#105
would you be open to a pr removing it from the wasm section or just adding a disclaimer saying that it is potentially broken?

#[global_allocator]
static ALLOC: wee_alloc::WeeAlloc = wee_alloc::WeeAlloc::INIT;

#[wasm_bindgen]
pub fn run() {
    App::new()
        .add_plugins(DefaultPlugins)
        .run();
}
@inodentry
Copy link
Contributor

OOF. OK, I will remove it.

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

No branches or pull requests

2 participants