32blit-sdk icon indicating copy to clipboard operation
32blit-sdk copied to clipboard

Remove decode_jpeg_* APIs?

Open Daft-Freak opened this issue 1 year ago • 2 comments
trafficstars

I've been thinking about this a bit:

  • As far as I can tell, decode_jpeg_buffer has two users (the example and my video player demo), decode_jpeg_file has none
  • They're a big chunk of code in the STM32 firmware (~6k text, ~5k bss)
  • Not even implemented in 32blit-pico
  • sdl2-image keeps causing dependency issues
  • It would probably be more useful to have something that wraps the same decoder everywhere on the user side of the API :thinking:

Daft-Freak avatar Sep 20 '24 11:09 Daft-Freak

I wonder if anyone else has used this? JPEG is definitely one of the worst ways to get image data onto a microcontroller and we have an explicit pipeline for converting assets!

Gadgetoid avatar Sep 20 '24 14:09 Gadgetoid

I would much rather see that valuable firmware space used for something that is widely useful and benefits from being fast, like more drawing routines (#32), higher level multiplayer code, or more flexible menu system, or maybe even something like stagelib. Even just another font or some standard button sprites would be more useful.

ali1234 avatar Sep 21 '24 00:09 ali1234