Skip to content

file: embed polyfill for blob and file#428

Merged
saghul merged 4 commits intosaghul:masterfrom
jimmywarting:files
Nov 14, 2023
Merged

file: embed polyfill for blob and file#428
saghul merged 4 commits intosaghul:masterfrom
jimmywarting:files

Conversation

@jimmywarting
Copy link
Copy Markdown
Contributor

@jimmywarting jimmywarting commented Nov 13, 2023

This is a kind of better blob implementation cuz it can references other blobs and dose not have to hold all data in memory.
it's a way better implementation for later being able to create blob's that are backed up by the disk.

it mostly comes from fetch-blob with some minor adjustments

@saghul
Copy link
Copy Markdown
Owner

saghul commented Nov 13, 2023

Thank you! Can you regenerate the C file please?

@jimmywarting
Copy link
Copy Markdown
Contributor Author

Plz Tell me how. This is my first contribution.

@saghul
Copy link
Copy Markdown
Owner

saghul commented Nov 13, 2023

Ssure, just run make js

@jimmywarting
Copy link
Copy Markdown
Contributor Author

Thx, Done!

@saghul
Copy link
Copy Markdown
Owner

saghul commented Nov 13, 2023

Looks like a test is failing, can you check?

@jimmywarting
Copy link
Copy Markdown
Contributor Author

jimmywarting commented Nov 13, 2023

A blob where constructed with invalid arguments in test-ws-send.js:13

- new Blob(arrayBuffer)
+ new Blob([ arrayBuffer ])

@saghul saghul merged commit 0551d36 into saghul:master Nov 14, 2023
@saghul
Copy link
Copy Markdown
Owner

saghul commented Nov 14, 2023

Thanks a lot!

@jimmywarting jimmywarting deleted the files branch November 14, 2023 21:13
@saghul
Copy link
Copy Markdown
Owner

saghul commented Nov 25, 2023

I was thinking about using your FormData libary to polyfill FormData, what do you think?

@jimmywarting
Copy link
Copy Markdown
Contributor Author

Sure. Feel free to copy as much or as little as you want

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.

2 participants