RomPatcher.js icon indicating copy to clipboard operation
RomPatcher.js copied to clipboard

BPS Linear Patch creation - Infinite Loop on BPS_getVLVLen

Open lilo-san opened this issue 2 years ago • 4 comments

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 avatar Apr 02 '23 19:04 lilo-san

Any thoughts on this would be appreciated.

lilo-san avatar Apr 05 '23 11:04 lilo-san

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 avatar Apr 07 '23 11:04 marcrobledo

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

lilo-san avatar Apr 07 '23 11:04 lilo-san

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

marcrobledo avatar Apr 07 '23 16:04 marcrobledo