Commit b1d8946
refactor: Replace Trove PrimeFinder with io.deephaven.hash.PrimeFinder
Swap gnu.trove.impl.PrimeFinder for io.deephaven.hash.PrimeFinder
(from libs.deephaven.hash, already on the compile classpath of both
Base and Util) in the four files that used it:
Base/cache/KeyedObjectCache.java
Base/cache/OpenAddressedCanonicalizationCache.java
Util/datastructures/WeakIdentityHashSet.java
Util/datastructures/hash/HashMapBase.java
PrimeFinder.nextPrime(int) is the only API used and has identical
semantics in both implementations, so this is a pure import swap.
Existing user io.deephaven.engine.table.impl.by.HashedRunFinder
already pulls from io.deephaven.hash.PrimeFinder; this commit
brings the four remaining call sites into line.
libs.trove is left in place on Base and Util because RetentionCache
(Constants, TObjectIntCustomHashMap, IdentityHashingStrategy) and
HashMapBase (TLongLongMap, TLongCollection, etc.) still reference
other Trove types.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>1 parent 533617c commit b1d8946
4 files changed
Lines changed: 5 additions & 5 deletions
File tree
- Base/src/main/java/io/deephaven/base/cache
- Util/src/main/java/io/deephaven/util/datastructures
- hash
Lines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
4 | 4 | | |
5 | 5 | | |
6 | 6 | | |
7 | | - | |
8 | 7 | | |
9 | | - | |
| 8 | + | |
| 9 | + | |
10 | 10 | | |
11 | 11 | | |
12 | 12 | | |
| |||
Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
5 | 5 | | |
6 | 6 | | |
7 | 7 | | |
8 | | - | |
| 8 | + | |
9 | 9 | | |
10 | 10 | | |
11 | 11 | | |
| |||
Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
3 | 3 | | |
4 | 4 | | |
5 | 5 | | |
6 | | - | |
| 6 | + | |
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
| |||
Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
6 | 6 | | |
7 | 7 | | |
8 | 8 | | |
9 | | - | |
10 | 9 | | |
| 10 | + | |
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
| |||
0 commit comments