Skip to content

Commit e42aeff

Browse files
committed
Squash some SpotBugs warnings
1 parent e929d4a commit e42aeff

File tree

5 files changed

+13
-0
lines changed

5 files changed

+13
-0
lines changed

src/main/java/com/rabbitmq/stream/codec/QpidProtonMessageBuilder.java

+2
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@
1818

1919
import com.rabbitmq.stream.Message;
2020
import com.rabbitmq.stream.MessageBuilder;
21+
import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
2122
import java.math.BigDecimal;
2223
import java.util.Date;
2324
import java.util.LinkedHashMap;
@@ -70,6 +71,7 @@ public Message build() {
7071
}
7172

7273
@Override
74+
@SuppressFBWarnings({"AT_NONATOMIC_64BIT_PRIMITIVE", "AT_STALE_THREAD_WRITE_OF_PRIMITIVE"})
7375
public MessageBuilder publishingId(long publishingId) {
7476
this.publishingId = publishingId;
7577
this.hasPublishingId = true;

src/main/java/com/rabbitmq/stream/codec/SimpleCodec.java

+2
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@
1818
import com.rabbitmq.stream.Message;
1919
import com.rabbitmq.stream.MessageBuilder;
2020
import com.rabbitmq.stream.Properties;
21+
import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
2122
import java.util.Map;
2223
import java.util.concurrent.atomic.AtomicBoolean;
2324

@@ -107,6 +108,7 @@ public Message build() {
107108
}
108109

109110
@Override
111+
@SuppressFBWarnings({"AT_NONATOMIC_64BIT_PRIMITIVE", "AT_STALE_THREAD_WRITE_OF_PRIMITIVE"})
110112
public MessageBuilder publishingId(long publishingId) {
111113
this.publishingId = publishingId;
112114
this.hasPublishingId = true;

src/main/java/com/rabbitmq/stream/codec/SwiftMqMessageBuilder.java

+2
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@
2121
import com.swiftmq.amqp.v100.generated.transport.definitions.SequenceNo;
2222
import com.swiftmq.amqp.v100.messaging.AMQPMessage;
2323
import com.swiftmq.amqp.v100.types.*;
24+
import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
2425
import java.io.IOException;
2526
import java.math.BigDecimal;
2627
import java.util.LinkedHashMap;
@@ -77,6 +78,7 @@ public Message build() {
7778
}
7879

7980
@Override
81+
@SuppressFBWarnings({"AT_NONATOMIC_64BIT_PRIMITIVE", "AT_STALE_THREAD_WRITE_OF_PRIMITIVE"})
8082
public MessageBuilder publishingId(long publishingId) {
8183
this.publishingId = publishingId;
8284
this.hasPublishingId = true;

src/main/java/com/rabbitmq/stream/codec/WrapperMessageBuilder.java

+2
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@
1818
import com.rabbitmq.stream.MessageBuilder;
1919
import com.rabbitmq.stream.Properties;
2020
import com.rabbitmq.stream.amqp.*;
21+
import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
2122
import java.math.BigDecimal;
2223
import java.util.Date;
2324
import java.util.LinkedHashMap;
@@ -56,6 +57,7 @@ public Message build() {
5657
}
5758

5859
@Override
60+
@SuppressFBWarnings({"AT_NONATOMIC_64BIT_PRIMITIVE", "AT_STALE_THREAD_WRITE_OF_PRIMITIVE"})
5961
public MessageBuilder publishingId(long publishingId) {
6062
this.publishingId = publishingId;
6163
this.hasPublishingId = true;

src/main/java/com/rabbitmq/stream/impl/StreamConsumerBuilder.java

+5
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@
1818
import static com.rabbitmq.stream.impl.Utils.SUBSCRIPTION_PROPERTY_MATCH_UNFILTERED;
1919

2020
import com.rabbitmq.stream.*;
21+
import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
2122
import java.lang.reflect.Field;
2223
import java.lang.reflect.Modifier;
2324
import java.time.Duration;
@@ -112,6 +113,7 @@ public ConsumerBuilder subscriptionListener(SubscriptionListener subscriptionLis
112113
}
113114

114115
@Override
116+
@SuppressFBWarnings("AT_STALE_THREAD_WRITE_OF_PRIMITIVE")
115117
public ManualTrackingStrategy manualTrackingStrategy() {
116118
this.manualTrackingStrategy = new DefaultManualTrackingStrategy(this);
117119
this.autoTrackingStrategy = null;
@@ -120,6 +122,7 @@ public ManualTrackingStrategy manualTrackingStrategy() {
120122
}
121123

122124
@Override
125+
@SuppressFBWarnings("AT_STALE_THREAD_WRITE_OF_PRIMITIVE")
123126
public AutoTrackingStrategy autoTrackingStrategy() {
124127
this.autoTrackingStrategy = new DefaultAutoTrackingStrategy(this);
125128
this.manualTrackingStrategy = null;
@@ -128,6 +131,7 @@ public AutoTrackingStrategy autoTrackingStrategy() {
128131
}
129132

130133
@Override
134+
@SuppressFBWarnings("AT_STALE_THREAD_WRITE_OF_PRIMITIVE")
131135
public ConsumerBuilder noTrackingStrategy() {
132136
this.noTrackingStrategy = true;
133137
this.autoTrackingStrategy = null;
@@ -140,6 +144,7 @@ public FlowConfiguration flow() {
140144
return this.flowConfiguration;
141145
}
142146

147+
@SuppressFBWarnings("AT_STALE_THREAD_WRITE_OF_PRIMITIVE")
143148
StreamConsumerBuilder lazyInit(boolean lazyInit) {
144149
this.lazyInit = lazyInit;
145150
return this;

0 commit comments

Comments
 (0)