ed/Odin
1
0
Fork 0
mirror of https://github.com/Ed94/Odin.git synced 2026-08-06 15:48:51 +00:00
Code Issues Packages Projects Releases Wiki Activity

container/queue: Deprecate peek_*

Browse Source 
The `*_ptr` and `peek_*` procedures did the same thing, except `peek_*`
was over-cautiously putting the index through a modulo when all
assignments to `q.offset` are already wrapped.
This commit is contained in:
Feoramund
2025-06-11 11:54:52 -04:00
parent 27cd508571
commit 58bda1209a
1 changed files with 4 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files
core/container/queue/queue.odin
+4 -6
View File
@@ -127,16 +127,14 @@ get_ptr :: proc(q: ^$Q/Queue($T), #any_int i: int, loc := #caller_location) -> ^
return &q.data[idx] return &q.data[idx]
} }
@(deprecated="Use `front_ptr` instead")
peek_front :: proc(q: ^$Q/Queue($T), loc := #caller_location) -> ^T { peek_front :: proc(q: ^$Q/Queue($T), loc := #caller_location) -> ^T {
runtime.bounds_check_error_loc(loc, 0, builtin.len(q.data)) return front_ptr(q, loc)
idx := q.offset%builtin.len(q.data)
return &q.data[idx]
} }
@(deprecated="Use `back_ptr` instead")
peek_back :: proc(q: ^$Q/Queue($T), loc := #caller_location) -> ^T { peek_back :: proc(q: ^$Q/Queue($T), loc := #caller_location) -> ^T {
runtime.bounds_check_error_loc(loc, int(q.len - 1), builtin.len(q.data)) return back_ptr(q, loc)
idx := (uint(q.len - 1)+q.offset)%builtin.len(q.data)
return &q.data[idx]
} }
// Push an element to the back of the queue // Push an element to the back of the queue