devinacker / bsnes-plus

debug-oriented fork of bsnes
http://bsnes.revenant1.net
327 stars 92 forks source link

IPS Selection #186

Open torridgristle opened 6 years ago

torridgristle commented 6 years ago

As far as I can tell bsnes+ will only apply an IPS patch if it has the same filename as the ROM, it'd be useful if you could load the ROM and then choose an arbitrarily named IPS patch from the file browser. Also the ability to apply multiple patches in the order they were loaded in, as opposed to making an IPS patch for any number of combinations of options and then renaming it to match the ROM each time you want to test something else. Multiple versions of a patch, multiple patches what do different things, etc.

devinacker commented 6 years ago

While this would be useful, there are a couple of things that make me a little unsure about actually implementing this:

Would it be integrated directly into the default file browser? If so, then it'd be totally inaccessible to anyone using the OS-native common file dialogs instead. Otherwise if it was presented separately (e.g. like the dialogs for loading slotted carts) it seems like it could end up being an extra step when loading ROMs that most people wouldn't really care about most of the time.

Also, from an end-user standpoint, I'd wager nearly all instances of people using soft patching when loading ROMs is for playing distributable hacks/translations/etc without needing any extra tools. If you're dealing with this level of complexity with multiple patches etc. then that's probably better suited to an actual IPS patcher or some other kind of external tool.

torridgristle commented 6 years ago

I figure loading an IPS patch could be done a few ways:

As for stacking patches, I reckon a pop-up window asking if it should replace or append the current patch would work fine for all methods except the command line, which should use all the patches specified in the order given.