Skip to content

Sokol Renderer: Fix Warning #538

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 20 commits into
base: main
Choose a base branch
from
Open

Sokol Renderer: Fix Warning #538

wants to merge 20 commits into from

Conversation

@timwmillard
Copy link

@timwmillard timwmillard commented Oct 10, 2025

Clay_RenderCommandArray uses int32_t not uint32_t.

Currently getting this warning on clang:

sokol_clay.h:332:28: warning: comparison of integers of different signs: 'uint32_t' (aka 'unsigned int') and 'int32_t' (aka 'int') [-Wsign-compare]
    for (uint32_t i = 0; i < renderCommands.length; i++) {

timwmillard and others added 20 commits July 11, 2025 01:17
@timwmillard
fix: for loop to use int32_t
a5895b2
@nicbarker @timwmillard
[Core] Split base ID hash from index ID hash
0b642e1
@github-actions @timwmillard
[bindings/odin] Update Odin bindings
a1b98c8
@Daniel-Shunom @timwmillard
[Documentation] -- updated README (nicbarker#475)
20610db
@nicbarker @timwmillard
[Core] Increase the default scroll container limit to 100
12850f4
@github-actions @timwmillard
[bindings/odin] Update Odin bindings
c9d29c5
@andrewmd5 @timwmillard
[Renderers/Sokol] Text measurement/rendering width mismatch in sokol_…
4878065 
…clay (nicbarker#480)
@rivten @andrewmd5 @timwmillard
[Renderers/Sokol] allow usage of images with sokol renderer (nicbarke…
5545cd8 
…r#489)

Co-authored-by: Hugo Viala <[email protected]>
Co-authored-by: Andrew Sampson <[email protected]>
@fgungor @timwmillard
[Core] fix unused extra macro param in CLAY_SID_LOCAL (nicbarker#505)
2bb60c4
@github-actions @timwmillard
[bindings/odin] Update Odin bindings
33f0ff7
@rats159 @timwmillard
[Bindings/Odin] Fix ID procedure ignoring index (nicbarker#496)
13f9fa6
@victor-Lopez25 @timwmillard
[Compilers] Fix struct with default initializer not using CLAY__DEFAU…
4f1726d 
…LT_STRUCT (nicbarker#498)
@github-actions @timwmillard
[bindings/odin] Update Odin bindings
ebe9b2e
@nicbarker @timwmillard
[Core] Remove ID from element declaration struct and split clay macro…
615279a 
… into CLAY and CLAY_WITHID (nicbarker#492)
@github-actions @timwmillard
[bindings/odin] Update Odin bindings
d310baf
@nicbarker @timwmillard
Fix README update
6c6489e
@elmfrain @nicbarker @timwmillard
[Bug Fix] Multiple Floating Elements Cannot Use Clay_Hovered() (nic…
35ad846 
…barker#461)

Co-authored-by: Nic Barker <[email protected]>
@github-actions @timwmillard
[bindings/odin] Update Odin bindings
191f58b
@Godje @timwmillard
[Renderers/termbox] Fixing termbox2-image-demo build error (nicbarker…
9495257 
…#524)
@timwmillard
Merge branch 'main' into sokol-fixes
6d08f68
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

10 participants

@timwmillard @nicbarker @Daniel-Shunom @andrewmd5 @rivten @fgungor @rats159 @victor-Lopez25 @elmfrain @Godje