Skip to content
This repository was archived by the owner on Feb 14, 2025. It is now read-only.

Fix JdkObsolete warnings #30

Open
mfvanek opened this issue Jun 1, 2023 · 0 comments
Open

Fix JdkObsolete warnings #30

mfvanek opened this issue Jun 1, 2023 · 0 comments
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@mfvanek
Copy link
Owner

mfvanek commented Jun 1, 2023

There are some warnings like:

LFUCache.java:94: warning: [JdkObsolete] It is very rare for LinkedList to out-perform ArrayList or ArrayDeque. Avoid it unless you're willing to invest a lot of time into benchmarking. Caveat: LinkedList supports null elements, but ArrayDeque does not.
        final List<Map.Entry<K, V>> evictedItems = new LinkedList<>();
                                                   ^
    (see https://errorprone.info/bugpattern/JdkObsolete)
  Did you mean 'final List<Map.Entry<K, V>> evictedItems = new ArrayList<>();'?

Need to fix them all

@mfvanek mfvanek added enhancement New feature or request help wanted Extra attention is needed good first issue Good for newcomers labels Jun 1, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

1 participant