Skip to content

Commit 4645ce6

Browse files
committed
Polishing
1 parent 564d3f6 commit 4645ce6

File tree

3 files changed

+11
-14
lines changed

3 files changed

+11
-14
lines changed

Diff for: spring-core/src/main/java/org/springframework/objenesis/SpringObjenesis.java

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/*
2-
* Copyright 2002-2021 the original author or authors.
2+
* Copyright 2002-2025 the original author or authors.
33
*
44
* Licensed under the Apache License, Version 2.0 (the "License");
55
* you may not use this file except in compliance with the License.
@@ -69,7 +69,7 @@ public SpringObjenesis(InstantiatorStrategy strategy) {
6969
this.strategy = (strategy != null ? strategy : new StdInstantiatorStrategy());
7070

7171
// Evaluate the "spring.objenesis.ignore" property upfront...
72-
if (SpringProperties.getFlag(SpringObjenesis.IGNORE_OBJENESIS_PROPERTY_NAME)) {
72+
if (SpringProperties.getFlag(IGNORE_OBJENESIS_PROPERTY_NAME)) {
7373
this.worthTrying = Boolean.FALSE;
7474
}
7575
}

Diff for: spring-jms/src/main/java/org/springframework/jms/config/AbstractJmsListenerContainerFactory.java

+2-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/*
2-
* Copyright 2002-2023 the original author or authors.
2+
* Copyright 2002-2025 the original author or authors.
33
*
44
* Licensed under the Apache License, Version 2.0 (the "License");
55
* you may not use this file except in compliance with the License.
@@ -209,6 +209,7 @@ public void setObservationRegistry(ObservationRegistry observationRegistry) {
209209
this.observationRegistry = observationRegistry;
210210
}
211211

212+
212213
@Override
213214
public C createListenerContainer(JmsListenerEndpoint endpoint) {
214215
C instance = createContainerInstance();

Diff for: spring-jms/src/main/java/org/springframework/jms/listener/AbstractMessageListenerContainer.java

+7-11
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/*
2-
* Copyright 2002-2023 the original author or authors.
2+
* Copyright 2002-2025 the original author or authors.
33
*
44
* Licensed under the Apache License, Version 2.0 (the "License");
55
* you may not use this file except in compliance with the License.
@@ -102,7 +102,7 @@
102102
* (i.e. after your business logic executed but before the JMS part got committed),
103103
* so duplicate message detection is just there to cover a corner case.
104104
* <li>Or wrap your <i>entire processing with an XA transaction</i>, covering the
105-
* reception of the JMS message as well as the execution of the business logic in
105+
* receipt of the JMS message as well as the execution of the business logic in
106106
* your message listener (including database operations etc). This is only
107107
* supported by {@link DefaultMessageListenerContainer}, through specifying
108108
* an external "transactionManager" (typically a
@@ -152,7 +152,8 @@ public abstract class AbstractMessageListenerContainer extends AbstractJmsListen
152152
implements MessageListenerContainer {
153153

154154
private static final boolean micrometerJakartaPresent = ClassUtils.isPresent(
155-
"io.micrometer.jakarta9.instrument.jms.JmsInstrumentation", AbstractMessageListenerContainer.class.getClassLoader());
155+
"io.micrometer.jakarta9.instrument.jms.JmsInstrumentation",
156+
AbstractMessageListenerContainer.class.getClassLoader());
156157

157158
@Nullable
158159
private volatile Object destination;
@@ -170,14 +171,14 @@ public abstract class AbstractMessageListenerContainer extends AbstractJmsListen
170171
@Nullable
171172
private String subscriptionName;
172173

174+
private boolean pubSubNoLocal = false;
175+
173176
@Nullable
174177
private Boolean replyPubSubDomain;
175178

176179
@Nullable
177180
private QosSettings replyQosSettings;
178181

179-
private boolean pubSubNoLocal = false;
180-
181182
@Nullable
182183
private MessageConverter messageConverter;
183184

@@ -500,12 +501,7 @@ public void setReplyPubSubDomain(boolean replyPubSubDomain) {
500501
*/
501502
@Override
502503
public boolean isReplyPubSubDomain() {
503-
if (this.replyPubSubDomain != null) {
504-
return this.replyPubSubDomain;
505-
}
506-
else {
507-
return isPubSubDomain();
508-
}
504+
return (this.replyPubSubDomain != null ? this.replyPubSubDomain : isPubSubDomain());
509505
}
510506

511507
/**

0 commit comments

Comments
 (0)