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

Update Enter the Matrix Widescreen Fix for Patch v1.2 #1648

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

SiliconExarch
Copy link

@SiliconExarch SiliconExarch commented Oct 21, 2024

As discussed in #774, patch 1.52 is not the newest available update for Enter The Matrix as Patch v1.2, which is 6 months newer than 1.52 and 2 months newer than the executable from the DVD-ROM Edition is available here.

This pull request updates the Widescreen Fix, accounting for the fact that depending on whether you install the CD Edition then update to 1.52 followed by v1.2 or install v1.2 on top of the DVD Edition you may have one of two different versions of MatrixConfig.exe - in the end I was able to handle this without having to make separate versions of the fix for each scenario.

@ThirteenAG
Copy link
Owner

Crashes with the version I have from back in the day:

image

@SiliconExarch
Copy link
Author

It's not expected to work for patch 1.52 anymore because the DVD-ROM edition and subsequent versions have significant code changes to the area in question on account of adding support for some additional (non-widescreen) resolutions.

@ThirteenAG
Copy link
Owner

Then just split the code with if else. If pattern is found, patch it, else find another pattern and patch it.

@ThirteenAG
Copy link
Owner

Alternatively you could just copy and paste the existing code to a separate RegisterCallback call, just make sure patterns are different in those and correspond with an appropriate exe.

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