Skip to content

Commit b502ea6

Browse files
committed
Remove Collections#unmodifiableMap() calls
Use methods from Map instead.
1 parent 0a11871 commit b502ea6

File tree

2 files changed

+10
-12
lines changed

2 files changed

+10
-12
lines changed

src/main/java/com/rabbitmq/client/amqp/ByteCapacity.java

Lines changed: 9 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,6 @@
1717
1818
package com.rabbitmq.client.amqp;
1919

20-
import java.util.Collections;
21-
import java.util.HashMap;
2220
import java.util.Map;
2321
import java.util.Objects;
2422
import java.util.function.BiFunction;
@@ -43,15 +41,15 @@ public class ByteCapacity implements Comparable<ByteCapacity> {
4341
private static final String UNIT_TB = "tb";
4442

4543
private static final Map<String, BiFunction<Long, String, ByteCapacity>> CONSTRUCTORS =
46-
Collections.unmodifiableMap(
47-
new HashMap<String, BiFunction<Long, String, ByteCapacity>>() {
48-
{
49-
put(UNIT_KB, (size, input) -> ByteCapacity.kB(size, input));
50-
put(UNIT_MB, (size, input) -> ByteCapacity.MB(size, input));
51-
put(UNIT_GB, (size, input) -> ByteCapacity.GB(size, input));
52-
put(UNIT_TB, (size, input) -> ByteCapacity.TB(size, input));
53-
}
54-
});
44+
Map.of(
45+
UNIT_KB,
46+
ByteCapacity::kB,
47+
UNIT_MB,
48+
ByteCapacity::MB,
49+
UNIT_GB,
50+
ByteCapacity::GB,
51+
UNIT_TB,
52+
ByteCapacity::TB);
5553

5654
private final long bytes;
5755
private final String input;

src/main/java/com/rabbitmq/client/amqp/impl/AmqpManagement.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -537,7 +537,7 @@ private DefaultQueueInfo(Map<String, Object> response) {
537537
this.autoDelete = (Boolean) response.get("auto_delete");
538538
this.exclusive = (Boolean) response.get("exclusive");
539539
this.type = QueueType.valueOf(((String) response.get("type")).toUpperCase(Locale.ENGLISH));
540-
this.arguments = Collections.unmodifiableMap((Map<String, Object>) response.get("arguments"));
540+
this.arguments = Map.copyOf((Map<String, Object>) response.get("arguments"));
541541
this.leader = (String) response.get("leader");
542542
String[] replicas = (String[]) response.get("replicas");
543543
if (replicas == null || replicas.length == 0) {

0 commit comments

Comments
 (0)