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

BPS Linear Patch creation - Infinite Loop on BPS_getVLVLen #59

Open
lilo-san opened this issue Apr 2, 2023 · 4 comments
Open

BPS Linear Patch creation - Infinite Loop on BPS_getVLVLen #59

lilo-san opened this issue Apr 2, 2023 · 4 comments
Labels

Comments

@lilo-san
Copy link

lilo-san commented Apr 2, 2023

Dear Marc,

Please try to create a BPS patch from these two files: https://we.tl/t-sqb2zBb9KX

Original: Tomb Raider - The Last Revelation (USA) (Rev 1).bin
Final: Tomb Raider - The Last Revelation (USA) (Rev 1) - TR4 Improved [Fix by Farglior v1.0].bin

I know these are big files, but to me they just expose some underlining problem with the BPS linear patch creation algorithm that could happen with smaller ones.

Best,
Juan

@lilo-san
Copy link
Author

lilo-san commented Apr 5, 2023

Any thoughts on this would be appreciated.

@marcrobledo
Copy link
Owner

Looks like a big issue. Wish I had time for this at the moment :-(

Unfortunately, I'm just offering minimal manteinance to Rom Patcher JS, as I'm focused on other things.

@marcrobledo marcrobledo added the bug label Apr 7, 2023
@lilo-san
Copy link
Author

lilo-san commented Apr 7, 2023

Can you tell me from where did you copy the linear patch algorithm?

@marcrobledo
Copy link
Owner

Sure!
Got an old copy of the original BPS documentation. I tried to find more information, but all documents in the net are the same. Sadly, the BPS format author passed away some years ago :-( And I don't recall seeing other BPS creation implementations.

specification_bps.zip

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants