Source:
<code>public static Map<String, Supplier<JGitFileSystem>> create(JGitFileSystemProviderConfiguration config) { return Collections.synchronizedMap(new LinkedHashMap<String, Supplier<JGitFileSystem>>(config.getJgitFileSystemsInstancesCache() + 1, 0.75f, true) { private Integer removeEldestEntryIterations = 0; @Override public Supplier<JGitFileSystem> putIfAbsent(String key, Supplier<JGitFileSystem> value) { Supplier<JGitFileSystem> jGitFileSystemSupplier = super.putIfAbsent(key, value); if (size() > config.getJgitFileSystemsInstancesCache()) { fitListToCacheSize(); } return jGitFileSystemSupplier; } private void fitListToCacheSize() { List<String> itemsToRemove = new ArrayList<>(); int maxIterations = config.getJgitCacheOverflowCleanupSize(); for (int i = this.size() - 1; (i >= 0 && (this.size() - i < maxIterations)); i--) { Map.Entry<String, Supplier<JGitFileSystem>> entry = (Map.Entry) this.entrySet().toArray()[i]; JGitFileSystem targetFS = (JGitFileSystem) ((MemoizedFileSystemsSupplier) entry.getValue()).get(); if (!targetFS.hasBeenInUse()) { itemsToRemove.add(entry.getKey()); } } <START> if (!itemsToRemove.isEmpty()) { <END> itemsToRemove.forEach(item -> this.remove(item)); } } @Override public boolean removeEldestEntry(Map.Entry eldest) { if (removeEldestEntryIterations > config.getJgitRemoveEldestEntryIterations()) { removeEldestEntryIterations = 0; return false; } if (size() > config.getJgitFileSystemsInstancesCache()) { JGitFileSystem targetFS = (JGitFileSystem) ((MemoizedFileSystemsSupplier) eldest.getValue()).get(); if (targetFS.hasBeenInUse()) { removeEldestEntryIterations++; this.remove(eldest.getKey()); this.put((String) eldest.getKey(), (MemoizedFileSystemsSupplier) eldest.getValue()); return false; } else { return true; } } return false; } }); }</code><technical_language>isEmpty() test is redundant. call itemsToRemove.forEach(item -> thir.remove(item))) testing for emptiness. If empty item removed in forEach, is ok</technical_language>

Target:
public static Map<String, Supplier<JGitFileSystem>> create(JGitFileSystemProviderConfiguration config) { return Collections.synchronizedMap(new LinkedHashMap<String, Supplier<JGitFileSystem>>(config.getJgitFileSystemsInstancesCache() + 1, 0.75f, true) { private Integer removeEldestEntryIterations = 0; @Override public Supplier<JGitFileSystem> putIfAbsent(String key, Supplier<JGitFileSystem> value) { Supplier<JGitFileSystem> jGitFileSystemSupplier = super.putIfAbsent(key, value); if (size() > config.getJgitFileSystemsInstancesCache()) { fitListToCacheSize(); } return jGitFileSystemSupplier; } private void fitListToCacheSize() { List<String> itemsToRemove = new ArrayList<>(); int maxIterations = config.getJgitCacheOverflowCleanupSize(); Object[] entries = this.entrySet().toArray(); for (int i = this.size() - 1; (i >= 0 && (this.size() - i < maxIterations)); i--) { Map.Entry<String, Supplier<JGitFileSystem>> entry = (Map.Entry) entries[i]; JGitFileSystem targetFS = (JGitFileSystem) ((MemoizedFileSystemsSupplier) entry.getValue()).get(); if (!targetFS.hasBeenInUse()) { itemsToRemove.add(entry.getKey()); } } itemsToRemove.forEach(item -> this.remove(item)); } @Override public boolean removeEldestEntry(Map.Entry eldest) { if (removeEldestEntryIterations > config.getJgitRemoveEldestEntryIterations()) { removeEldestEntryIterations = 0; return false; } if (size() > config.getJgitFileSystemsInstancesCache()) { JGitFileSystem targetFS = (JGitFileSystem) ((MemoizedFileSystemsSupplier) eldest.getValue()).get(); if (targetFS.hasBeenInUse()) { removeEldestEntryIterations++; this.remove(eldest.getKey()); this.put((String) eldest.getKey(), (MemoizedFileSystemsSupplier) eldest.getValue()); return false; } else { return true; } } return false; } }); }