Skip to content
This repository was archived by the owner on Mar 18, 2022. It is now read-only.

enable inline wasm code and safe buffer #22

Closed
wants to merge 2 commits into from
Closed

Conversation

mpetrunic
Copy link
Member

  • works on opera and firefox but not on chrome

@CLAassistant
Copy link

CLAassistant commented Oct 22, 2019

CLA assistant check
All committers have signed the CLA.

@MaxGraey
Copy link
Contributor

Plz don't use sync WebAssembly.Module for wasm which size more than 4 kb. Possible solution: webpack/webpack#6475 (comment)

@mpetrunic
Copy link
Member Author

Plz don't use sync WebAssembly.Module for wasm which size more than 4 kb. Possible solution: webpack/webpack#6475 (comment)

Yeah we are discussing that in discord. It's annoying issue because your solution will make module loaded async which means we would have to rewrite half of lodestar to make it work on web as it uses sha256 in synchronous context

@mpetrunic mpetrunic closed this Dec 9, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants