zig icon indicating copy to clipboard operation
zig copied to clipboard

`ArrayHashMap` confusing documentation

Open MrBrN197 opened this issue 1 year ago • 1 comments

this documentation for Context in ArrayHashMap seems to suggest the eql function takes 2 args but the compiler complains that it needs 3 arguments. https://github.com/ziglang/zig/blob/6a65561e3e5f82f126ec4795e5cd9c07392b457b/lib/std/array_hash_map.zig#L70-L77

MrBrN197 avatar May 17 '24 01:05 MrBrN197

Looks like it should be pub fn eql(self, K, K, second_keys_index) bool judging from array_hash_map.StringContext.eql:

https://github.com/ziglang/zig/blob/6a65561e3e5f82f126ec4795e5cd9c07392b457b/lib/std/array_hash_map.zig#L41

squeek502 avatar May 17 '24 02:05 squeek502

Introduced in cf88cf2657d721c68055a284e8c498a18639f74c

NicoElbers avatar Dec 11 '24 15:12 NicoElbers