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

Provide a meaningfull name to loaded scripts. #276

Open
github-actions bot opened this issue Dec 4, 2023 · 1 comment
Open

Provide a meaningfull name to loaded scripts. #276

github-actions bot opened this issue Dec 4, 2023 · 1 comment
Labels
code:todo A TODO/FIXME comment in code.

Comments

@github-actions
Copy link

github-actions bot commented Dec 4, 2023

env.set(ctx, "session", session_var)?;
// TODO: Provide a meaningfull name to loaded scripts.
let closure = Closure::load_with_env(ctx, None, self.source.as_bytes(), env)?;
let ex = Executor::start(ctx, closure.into(), ());
Ok(ctx.state.registry.stash(&ctx, ex))
})?;


This issue was generated by todo-issue based on a TODO comment in dd2f80d.
@github-actions github-actions bot added the code:todo A TODO/FIXME comment in code. label Dec 4, 2023
Copy link
Author

github-actions bot commented Dec 4, 2023

Provide a meaningfull name to loaded scripts.

let asset = asset_server.store.assets.get(&cid).unwrap();
let source = &asset.data.cast_ref::<LuaScript>().source;
// TODO: Provide a meaningfull name to loaded scripts.
let closure =
Closure::load_with_env(ctx, None, source.as_bytes(), env)?;
compiled_scripts
.insert(cid, ctx.state.registry.stash(&ctx, closure));


This comment was generated by todo-issue based on a TODO comment in dd2f80d.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code:todo A TODO/FIXME comment in code.
Projects
None yet
Development

No branches or pull requests

0 participants