diff --git a/src/ImTools/ImTools.cs b/src/ImTools/ImTools.cs index 955cf72..7c52c0a 100644 --- a/src/ImTools/ImTools.cs +++ b/src/ImTools/ImTools.cs @@ -2108,7 +2108,6 @@ internal virtual StringBuilder ToMermaidString(StringBuilder s) => /// otherwise the result. Returns the Entry with the `hash` though it may be a `HashConflictingEntry` internal virtual Entry GetSurePresentEntry(int hash) => throw new InvalidOperationException("The sure present hash does not exist in the empty map"); - // todo: @wip remove hash from here for simplicity /// Returns the found entry with the same hash or the new map with added new entry. /// Note that the empty map will return the entry the same as if the entry was found - so the consumer should check for the empty map. /// Note that the method cannot return the `null` - when the existing entry is not found it will always be the new map with the added entry. @@ -5877,7 +5876,7 @@ public static ImHashMap AddOrUpdate(this ImHashMap map, K key, [MethodImpl((MethodImplOptions)256)] public static ImHashMap AddOrUpdateByReferenceEquals(this ImHashMap map, K key, V value) => map.AddOrUpdateEntryByReferenceEquals(EntryWithHash(key, value)); - + // todo: @feature we need and Add method to throw for the existing key, maybe based on GetOrAdd /// Add sure not present item, so before calling this method you may either check the map via GetEntryOrNull or be sure that the new added key is unique [MethodImpl((MethodImplOptions)256)] public static ImHashMap AddSureNotPresentEntry(this ImHashMap map, ImHashMapEntry newEntry) => @@ -6389,7 +6388,6 @@ public bool MoveNext() void IDisposable.Dispose() { } } - // todo: @wip for the ImHashMap /// Do something for each entry. /// The `parents` parameter allows to reuse the stack memory used for the traversal between multiple calls. /// So you may pass the empty `parents` into the first `Enumerate` and then keep passing the same `parents` into the subsequent calls