Skip to content

Emscripten: retry download on uncommon failure#3004

Merged
Ghabry merged 1 commit into
EasyRPG:masterfrom
aleck099:master
Jun 5, 2023
Merged

Emscripten: retry download on uncommon failure#3004
Ghabry merged 1 commit into
EasyRPG:masterfrom
aleck099:master

Conversation

@aleck099

Copy link
Copy Markdown

From ynoclient: ynoproject/ynoengine@095cf5b
TCP connections aren't reliable!

From ynoclient: ynoproject/ynoengine@095cf5b
TCP connections aren't reliable!
@ghost ghost requested a review from Ghabry May 14, 2023 14:07
@Ghabry

Ghabry commented May 14, 2023

Copy link
Copy Markdown
Member

Thanks for the patch.

A good test for this is likely simulating a bad wi-fi connection / mobile connection.

E.g. disconnect the WiFi and then reconnect it after a few seconds.

Will test this later :)

@ghost ghost added the Emscripten WebAssembly/JavaScript port for web browsers label May 24, 2023

@Ghabry Ghabry left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When the connection drops all 10 retries fire pretty fast and I can't find a simple solution for waiting. So lgtm for now and can be improved in a later version. Thanks.

@Ghabry Ghabry merged commit 6832809 into EasyRPG:master Jun 5, 2023
@Ghabry Ghabry added this to the 0.8.0.1 milestone Jun 5, 2023
sevenc-nanashi pushed a commit to sevenc-nanashi/easyrpg-player that referenced this pull request May 31, 2026
Emscripten: retry download on uncommon failure
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Emscripten WebAssembly/JavaScript port for web browsers

Development

Successfully merging this pull request may close these issues.

2 participants