Skip to content

This adds the rest of the emscripten macros that wrap javascript code…#6234

Merged
chrchr-github merged 2 commits into
cppcheck-opensource:mainfrom
dsamo:patch-1
Apr 5, 2024
Merged

This adds the rest of the emscripten macros that wrap javascript code…#6234
chrchr-github merged 2 commits into
cppcheck-opensource:mainfrom
dsamo:patch-1

Conversation

@dsamo

@dsamo dsamo commented Apr 5, 2024

Copy link
Copy Markdown
Contributor

… to emscripten.cfg

Namely:
EM_ASM_DOUBLE
EM_ASM_INT
EM_ASM_PTR
MAIN_THREAD_EM_ASM_DOUBLE
MAIN_THREAD_EM_ASM_INT
MAIN_THREAD_EM_ASM_PTR
MAIN_THREAD_ASYNC_EM_ASM

@firewave

firewave commented Apr 5, 2024

Copy link
Copy Markdown
Collaborator

Half of the actions are not being run. Something's weird going on here.

@dsamo

dsamo commented Apr 5, 2024

Copy link
Copy Markdown
Contributor Author

@firewave My changes do not seem related to the errors. Should I do something from my side?

@chrchr-github

Copy link
Copy Markdown
Collaborator

If you wish, you could add yourself to the AUTHORS file. That would trigger a rerun.

@dsamo

dsamo commented Apr 5, 2024

Copy link
Copy Markdown
Contributor Author

@chrchr-github done

@dsamo

dsamo commented Apr 5, 2024

Copy link
Copy Markdown
Contributor Author

There is now an error in coverage build due to 502 server error - Rebased to retrigger

dsamo added 2 commits April 5, 2024 16:10
… to emscripten.cfg

Namely:
EM_ASM_DOUBLE
EM_ASM_INT
EM_ASM_PTR
MAIN_THREAD_EM_ASM_DOUBLE
MAIN_THREAD_EM_ASM_INT
MAIN_THREAD_EM_ASM_PTR
MAIN_THREAD_ASYNC_EM_ASM
@firewave

firewave commented Apr 5, 2024

Copy link
Copy Markdown
Collaborator

There is now an error in coverage build due to 502 server error - Rebased to retrigger

That happens from time to time. We can just re-run that step when it happens. No need for a re-trigger.

@chrchr-github chrchr-github merged commit 2f0bb71 into cppcheck-opensource:main Apr 5, 2024
@dsamo dsamo deleted the patch-1 branch April 5, 2024 14:10
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