search

prime31 / zig-ecs

MIT License
251 stars 39 forks source link

Fix for Handles and EntityTraits unsigned assertion with the new std.builtin.Signedness #7

Closed mbcrocci closed 3 years ago

mbcrocci commented 3 years ago

Hi, wasn't able to compile with zig version 0.7.0+500fbdad5 due to a recent zig PR #6713 that changes std.builtin.Int to have a new signedness field.

I don't think this occured with version 0.7.0. Not sure if you want to support zig master or keep it synced to minor versions, so feel free to reject the PR.

prime31 commented 3 years ago

I am stuck on 0.7.0 for macOS ARM support for the time being. I pushed a temporary hack that should work on all 0.7+ versions though.

mbcrocci commented 3 years ago

I can confirm that you temporary hack did indeed work. If anything changes I'll let you know.