Skip to content

Fix vanilla Deku Nut upgrade bug#333

Merged
Kenix3 merged 2 commits intoHarbourMasters:developfrom
vaguerant:dekunutgrade
May 21, 2022
Merged

Fix vanilla Deku Nut upgrade bug#333
Kenix3 merged 2 commits intoHarbourMasters:developfrom
vaguerant:dekunutgrade

Conversation

@vaguerant
Copy link
Contributor

Prevents the flag from writing if the bugfix cvar is set. I'm pretty confident the original behavior is not relied upon anywhere else as the flag is only otherwise referenced in relation to the Forest Stage, never the Poacher's Saw.

Enabled by default because there are no benefits to the bug.

Fixes #330.

Copy link
Contributor

@Rozelette Rozelette left a comment

Choose a reason for hiding this comment

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

LGTM. I checked myself and this flag does not seem to be used anywhere else beside the upgrade

@Kenix3 Kenix3 merged commit a3cf852 into HarbourMasters:develop May 21, 2022
stratomaster64 pushed a commit to stratomaster64/Shipwright that referenced this pull request Jun 13, 2022
stratomaster64 pushed a commit to stratomaster64/Shipwright that referenced this pull request Jun 20, 2022
Sirius902 pushed a commit to Sirius902/Shipwright that referenced this pull request Jul 11, 2022
…smalldevmerge

Toir hopefullysmalldevmerge
Malkierian pushed a commit to Malkierian/Shipwright that referenced this pull request Nov 20, 2023
* Fixed window pos not restored with OpenGL

* Fixed fullscreen in OpenGL (again)

Fixes going full screen on the next monitor when the game was on the second half of the screen.

* Set Fullscreen state only when call succeeds

Also some logging so we hopefully know why it failed.

* Consistent behavior for getting monitor

SDL always gets the monitor  at the middle of the window. Now DXGI does too. Also only resets the position if there is no pixel of the window on any screen.

* also get current monitor on resize (DirectX)

* Correct typo

Co-authored-by: Kenix3 <kenixwhisperwind@gmail.com>

---------

Co-authored-by: Kenix3 <kenixwhisperwind@gmail.com>
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.

Fix the vanilla Deku Nut upgrade bug

3 participants