Skip to content

Commit 3fa278b

Browse files
committed
Fix new Sonar smells
1 parent b54f830 commit 3fa278b

File tree

2 files changed

+8
-8
lines changed

2 files changed

+8
-8
lines changed

spring-amqp/src/main/java/org/springframework/amqp/core/Queue.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -176,9 +176,9 @@ public final void setMasterLocator(@Nullable String locator) {
176176
}
177177

178178
@Override
179-
public Object clone() { // NOSONAR
180-
Queue queue = new Queue(this.name, this.durable, this.exclusive, this.autoDelete,
181-
new HashMap<>(this.arguments));
179+
public Object clone() {
180+
Queue queue = new Queue(this.name, this.durable, this.exclusive, // NOSONAR - doesn't need to call super.clone()
181+
this.autoDelete, new HashMap<>(this.arguments));
182182
queue.setActualName(this.actualName);
183183
return queue;
184184
}

spring-rabbit/src/main/java/org/springframework/amqp/rabbit/listener/SimpleMessageListenerContainer.java

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -217,14 +217,14 @@ public void setConcurrency(String concurrency) {
217217
try {
218218
int separatorIndex = concurrency.indexOf('-');
219219
if (separatorIndex != -1) {
220-
int consumers = Integer.parseInt(concurrency.substring(0, separatorIndex));
221-
int maxConsumers = Integer.parseInt(concurrency.substring(separatorIndex + 1));
222-
Assert.isTrue(maxConsumers >= consumers,
220+
int consumersToSet = Integer.parseInt(concurrency.substring(0, separatorIndex));
221+
int maxConsumersToSet = Integer.parseInt(concurrency.substring(separatorIndex + 1));
222+
Assert.isTrue(maxConsumersToSet >= consumersToSet,
223223
"'maxConcurrentConsumers' value must be at least 'concurrentConsumers'");
224224
this.concurrentConsumers = 1;
225225
this.maxConcurrentConsumers = null;
226-
setConcurrentConsumers(consumers);
227-
setMaxConcurrentConsumers(maxConsumers);
226+
setConcurrentConsumers(consumersToSet);
227+
setMaxConcurrentConsumers(maxConsumersToSet);
228228
}
229229
else {
230230
setConcurrentConsumers(Integer.parseInt(concurrency));

0 commit comments

Comments
 (0)