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

example/wasm: fix the memory leak: wasm_buf #20200

Closed
wants to merge 2 commits into from

Conversation

zengwei00
Copy link
Contributor

fix the Memory leak: wasm_buf

Signed-off-by: zengwei zengwei1@uniontech.com

Contribution description

Testing procedure

Issues/PRs references

fix the Memory leak: wasm_buf

Signed-off-by: zengwei zengwei1@uniontech.com
@github-actions github-actions bot added the Area: examples Area: Example Applications label Dec 20, 2023
@kfessel kfessel added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Dec 20, 2023
@riot-ci
Copy link

riot-ci commented Dec 20, 2023

Murdock results

✔️ PASSED

dbd24db Merge branch 'RIOT-OS:master' into patch-1

Success Failures Total Runtime
12 0 12 01m:10s

Artifacts

@kfessel kfessel changed the title fix the Memory leak: wasm_buf example/wasm: fix the memory leak: wasm_buf Dec 20, 2023
@kfessel
Copy link
Contributor

kfessel commented Dec 20, 2023

change looks good

would you please prefix you commit the area the change is happening in -like i did with the title of this PR)

@zengwei00
Copy link
Contributor Author

Okay, I will close this PR and resubmit

@zengwei00 zengwei00 closed this Dec 20, 2023
@zengwei00
Copy link
Contributor Author

改变看起来不错

请您在提交发生更改的区域前加上前缀 - 就像我对此 PR 的标题所做的那样)

#20205 I resubmitted a PR

@kfessel
Copy link
Contributor

kfessel commented Dec 20, 2023

Okay, I will close this PR and resubmit

why didn't you just amend and (force) push to the already exiting commit?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: examples Area: Example Applications CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants