libretro / snes9x

Snes9x - Portable Super Nintendo Entertainment System (TM) emulator
http://www.snes9x.com
Other
46 stars 55 forks source link

Add 12-pixel and 16-pixel Crop Overscan options #258

Closed Ryunam closed 3 years ago

Ryunam commented 3 years ago

This PR introduces a change that takes inspiration from the bsnes-hd and Mesen-S cores.

In its current form, the snes9x libretro implementation allows users to define in the core options whether they want to crop the top and bottom lines of the picture by about 8 pixels, but there are quite a few cases where having a 12-pixel and a 16-pixel crop options might be desirable. Namely:

With this PR the core options now allow to select also a 12-pixel or a 16-pixel crop, based on one's preference. The existing configurations / overrides set by users prior to this implementation should not be altered.

A few pictures for comparison's sake:

Yoshi's Island - No cropping Super Mario - Yossy Island (Japan)-210516-195956

Yoshi's Island - Standard crop (about 8 pixels) Super Mario - Yossy Island (Japan)-210516-195959

_Yoshi's Island - 12-pixel crop Super Mario - Yossy Island (Japan)-210516-200001

_Yoshi's Island - 16-pixel crop Super Mario - Yossy Island (Japan)-210516-200003


The modifications applied with this PR are also upstream-friendly, in the sense that the only parts of the code that were altered are libretro-related.

See also this thread by @Awakened0 on the upstream bsnes repo for additional reference: https://github.com/bsnes-emu/bsnes/issues/169