search

libretro / beetle-pcfx-libretro

Standalone port of Mednafen PCFX to libretro.
GNU General Public License v2.0
9 stars 30 forks source link

Add support for 16 bpp color format #85

Closed negativeExponent closed 3 years ago

negativeExponent commented 3 years ago

32bpp color is still set as default. To build 16 bpp, just add NEED_BPP=16 when compiling. FRONTEND_SUPPORTS_RGB should also be set to 1.

Will the core be faster than 32bpp? dunno, don't have a proper way to test it.

@twinaphex @hunterk @hizzlekizzle I'll let you guys decide if we want 16bpp to be set as the default.

inactive123 commented 3 years ago

Since it's now a runtime option (I assume?), it probably makes sense to set 16bpp as the runtime default and then allow the user to still set it to 32bpp manually by changing the core option if they want.

negativeExponent commented 3 years ago

i haven't made it a runtime option (or selectable core option). ill do that then if its more ideal.

Since it's now a runtime option (I assume?), it probably makes sense to set 16bpp as the runtime default and then allow the user to still set it to 32bpp manually by changing the core option if they want.

inactive123 commented 3 years ago

Well, in doing so, do test if this comes at a performance cost. If it does, it might make sense to leave it as a compile time option and then just enable it for the platforms where we need slightly better performance.

In any case, I will leave these decisions entirely up to you.