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

Resync with master #54

Open
wants to merge 443 commits into
base: master
Choose a base branch
from
Open

Conversation

Desdaemon
Copy link
Contributor

No description provided.

rohkea and others added 30 commits June 24, 2024 21:28
Config for Soundfont, Font and "Focus Lost"
ttyp0 changes (remove Thai and Hebrew), generator fix
Removing debug code, polishing code

+ Added support for .relative and .centered options for Absolute Pixel pan. Needed to bitmask parameter[4] in order to check for those.

= Renamed a few commands
= Added comments for some functions
- Removed debug code

Adjusting the speed formula

Now it works very well, even has differnet speeds for both H and V. However since Maniac internally uses doubles for screen panning, we might have to do some more work...

Revert "Adjusting the speed formula"

This reverts commit 52fa8429ff463989c70297989a01aeab12238f1b.

Adjusting the speed formula Again

Now everything works as it should.

Time to use doubles for everything

Corrected interpolation, and proper scrolling speed

Correct formula for absolute positioning

Absolute Cam Panning now works

Changed some naming scheme

First things to do for Pixel Scrolling

Adding support for commands

However scrolling is a bit jaggy compared to ManiacPatch
- Implemented template method "DecodeTargetEvaluationMode".
  (This logic is shared between ControlVariables, ControlSwitches & ControlStrings but the actual implementation for several special access modes differs between implementations. Support for these modes is evaluated at compile time.)
- Moved some common helper functions into a new file "game_interpreter_shared.cpp" (CheckOperator, ValueOrVariable / CommandStringOrVariable & variants)
- Refactored "ValueOrVariable" & "ValueOrVariableBitfield" into template methods
- Moved some commonly used member function declarations which are needed for certain command argument evaluation methods from Game_Interpreter into a new abstract base class
  (namespace "Game_Interpreter_Shared" is agnostic to the actual implementation of the interpreter)
…d definition of "Game_BaseInterpreterContext" to the bottom of header file
…implicit instantiations here while GCC fails
From Maniac Patch version 240423:
[RPG_RT.exe]
Support for different magnification rates for special drawing of pictures (rotation, waveform, angle).

+ Added check if sprite's width/height is at zero. If so, skip the draw routine (helps avoid a glitchy sprite at the top-left when rotating an infinitely thin picture)
- Removed check for effect mode when setting zoom or width/height.

One thing to note when comparing Maniac and EasyRPG: the former doesn't seem to properly clip the sprite in the corners, when rotating the sprite.
In Maniacs, CommandOpenSaveMenu is renamed to "Call System Function".
I implemented what is closer to our code.

Also expanded the code to cover our custom submenus (Cases 8 to 12).

Update game_interpreter_map.cpp
…hen the event vector is altered.

Renamed to CloneMapEvent
…t running is destroyed.

Prevents triggering the sanity check about an invalid event on the main interpreter.

Fix operator= of Game_CharacterDataStorage.
If you have a copy of config.ini saved at the same folder as the player app, it will load the game settings from it, instead of the global folder.

-------------------------------------
Update game_config.cpp
Otherwise, it will complain about undefined references to GetSdlAxis.
Fix compiling OpenDingux code path with SDL 1.2
Android: Add LZH support, refactor GameBrowser Game Searching
aidatorajiro and others added 18 commits January 14, 2025 10:28
Otherwise the initialization is not applied when loading a Savegame

Fix #3327
Map: initialize screen width and height on SetupCommon
That solves some errors related out of bounds access.
Sometimes it happens when drawing a tile, others when checking passability.

It may differ from how maniacs deals with this (I suspect maniacs have an extra blank tile, with locked passability, just for OOB tiles).
Added comments to warn about not ever doing this again.

Fix #3333
The "Triggered By Decision Key" state is now forwarded from the source event.

Suggested by @florianessl
Make localization simpler by merging some strings
Add (Sim) RPG Maker 95 to detection
Our homebrew platforms have too slow IO for this.
@github-actions github-actions bot added documentation Improvements or additions to documentation Building labels Jan 21, 2025
Partial revert of e7767ff (the code that replaced min/max with clamp)
TriggerEventAt: Add flag to configure "Face Player" behaviour
Android: Add button to open save directory and better "open folder" experience
@Desdaemon Desdaemon removed documentation Improvements or additions to documentation Building labels Jan 25, 2025
@Ghabry
Copy link
Contributor

Ghabry commented Feb 22, 2025

I highly recommend updating when this is merged EasyRPG/Player#3343

Without this CU will crash when using various event commands because the argument list is shorter than expected

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.