zig/mach
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

object: fix Objects.Slice delete/get/set methods

Browse source 
Signed-off-by: Stephen Gutekanst <stephen@hexops.com>
This commit is contained in:
Stephen Gutekanst 2024-12-01 11:44:23 -07:00 • committed by Emi Gutekanst
parent 3aee6995d7
commit 58d9c378ca
1 changed files with 8 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

16
src/module.zig
View file

@ -88,28 +88,28 @@ pub fn Objects(options: ObjectsOptions, comptime T: type) type {
objs: *Objects(options, T), objs: *Objects(options, T),
/// Same as Objects(T).set but doesn't employ safety checks /// Same as Objects(T).set but doesn't employ safety checks
pub fn set(objs: *@This(), id: ObjectID, value: T) void { pub fn set(s: *@This(), id: ObjectID, value: T) void {
const data = &objs.internal.data; const data = &s.objs.internal.data;
const unpacked: PackedID = @bitCast(id); const unpacked: PackedID = @bitCast(id);
data.set(unpacked.index, value); data.set(unpacked.index, value);
} }
/// Same as Objects(T).get but doesn't employ safety checks /// Same as Objects(T).get but doesn't employ safety checks
pub fn get(objs: *@This(), id: ObjectID) ?T { pub fn get(s: *@This(), id: ObjectID) T {
const data = &objs.internal.data; const data = &s.objs.internal.data;
const unpacked: PackedID = @bitCast(id); const unpacked: PackedID = @bitCast(id);
return data.get(unpacked.index); return data.get(unpacked.index);
} }
/// Same as Objects(T).delete but doesn't employ safety checks /// Same as Objects(T).delete but doesn't employ safety checks
pub fn delete(objs: *@This(), id: ObjectID) void { pub fn delete(s: *@This(), id: ObjectID) void {
const dead = &objs.internal.dead; const dead = &s.objs.internal.dead;
const recycling_bin = &objs.internal.recycling_bin; const recycling_bin = &s.objs.internal.recycling_bin;
const unpacked: PackedID = @bitCast(id); const unpacked: PackedID = @bitCast(id);
if (recycling_bin.items.len < recycling_bin.capacity) { if (recycling_bin.items.len < recycling_bin.capacity) {
recycling_bin.appendAssumeCapacity(unpacked.index); recycling_bin.appendAssumeCapacity(unpacked.index);
} else objs.internal.thrown_on_the_floor += 1; } else s.objs.internal.thrown_on_the_floor += 1;
dead.set(unpacked.index); dead.set(unpacked.index);
} }