[TomJGooding]

I noticed a flurry of updates in the stb libraries recently - this updates those years old files in Nuklear.

I confirmed that our previous patch to stb_truetype.h in #905 has now been fixed upstream.

Resolves #953

[RobLoach]

Thanks, should this target 5.x branch?

[RobLoach]

Also, could there be a way for us to consolidate all the stb dependencies into one folder so it's easy to update down the line?

[TomJGooding]

We could move these to some thirdparty/vendor/dependencies folder - just let me know your preference for the name!

Does this need to target the 5.x branch? These files aren't compatible with c89 already, hence the script to remove the comments?

I haven't updated stb_image.h only because I wasn't sure where it belongs #953 (comment). But again, these files aren't compatible with c89 already.

EDIT: Sorry just to clarify: the reason for not simply updating these files when the issue was initially raised was about c89 compatibility, where we discussed needing some script for fetching the latest versions and removing C++ comments (and applying our patches). But since the relevant demos/examples are complied with -std=c99 already, and Nuklear soon will be moving away from c89 anyway, is there any reason not to update now?

[TomJGooding]

I realise that moving the stb libraries to some /thirdpaty folder would require a bit of rewiring (see #953 (comment)), but I'm more than happy to tackle this now rather than revisit later if it helps!

For now I've updated both copies of stb_image.h in example/ and demo/. Alternatively as suggested in #953 (comment), we could remove one and just point the other demo/examples to that version.

[TomJGooding]

There's also nuklear_text_editor.c which is a modified version of stb_textedit as pointed out in #953 (comment). That's also a very old version (v1.8 back in 2016), but obviously is less straightforward to update.