Immediate-Mode-UI / Immediate-Mode-UI/Nuklear

stb_image inconsistencies

Open
#953 12 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
C
Stars
11.4k
Forks
686
Avg merge
4d 1h
Merged PRs (30d)
3

Description

stb_image is used to load images in various `example/*` and `demo/*` examples.

There's currently two different stb_image versions, where `example/stb_image.h` is v2.08 and `demo/common/filebrowser/stb_image.h` is v2.28.

stb_image also isn't compatible with C89 because of the C++ style comments.

Would it make sense to create a single stb_image file with the latest version (2.30) that is compatible with C89?

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Inspect example/stb_image.h and demo/common/filebrowser/stb_image.h, then trace how the example/* and demo/* entry points include them. Compare the two copies with stb_image v2.30 and check its C89 compatibility, including the C++-style comments. Done means the examples use one consistent, C89-compatible stb_image header.

Written by the indexing model from the issue text.

Assessment

Tech stack
c
Domain
computer-graphics
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
58/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.