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

Emscripten Async #462

Merged
merged 29 commits into from May 2, 2015
Merged

Emscripten Async #462

merged 29 commits into from May 2, 2015

Conversation

@Ghabry
Copy link
Member

@Ghabry Ghabry commented May 1, 2015

This asynchifies the whole file loading code.

Start via
https://easy-rpg.org/play/index-pr.html?game=testgame-2000

In short this introduces a new class AsyncManager where file requests can be registred to. And when a file finishes downloading an EventHandler invokes that does the file access (e.g. assigning Bitmap or playing music).
On Desktop the event fires immediately so there is no change in code path compared to the old code.

At the moment I have debug code enabled that fakes async loading on Desktop, must be removed before merging.

This also adds a php script easyrpg-filefinder.php that handles the case insensitive file finding on server side.
Based on position of player.html place the games in "games/[gamename]", filefinder in "games". Needs a writable cache-dir at "games/cache" (For filelist)

TODO:

  • new Ib version @carstene1ns
  • Disable RTP not found warning
  • Use canvas zoom instead of SDL zoom
  • Document asyncManager file
  • Remove async debug code
Ghabry added 27 commits Feb 23, 2015
…remaining calls except ShopParty and Music/Sound
@Ghabry Ghabry force-pushed the Ghabry:emscripten_playground branch 3 times, most recently from b998841 to 8a7e8bd May 2, 2015
@Ghabry Ghabry force-pushed the Ghabry:emscripten_playground branch from 8a7e8bd to 554ad59 May 2, 2015
fdelapena added a commit that referenced this pull request May 2, 2015
@fdelapena fdelapena merged commit 25c89d0 into EasyRPG:master May 2, 2015
2 checks passed
2 checks passed
@fdelapena
default Merged build finished.
Details
@fdelapena
player-js-pr Build #145 succeeded in 1 min 27 sec
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants