fix(deps): update errorproneversion to v2.41.0 (#7512)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Jason Plumb <[email protected]>

Author: renovate[bot]

dependencyManagement/build.gradle.kts

@@ -28,7 +28,7 @@ val DEPENDENCY_BOMS = listOf(
 )
 
 val autoValueVersion = "1.11.0"
-val errorProneVersion = "2.40.0"
+val errorProneVersion = "2.41.0"
 val jmhVersion = "1.37"
 // Mockito 5.x.x requires Java 11 https://github.com/mockito/mockito/releases/tag/v5.0.0
 val mockitoVersion = "4.11.0"

exporters/common/src/main/java/io/opentelemetry/exporter/internal/marshal/UnsafeAccess.java

@@ -71,7 +71,7 @@ static long getLong(Object o, long offset) {
   private UnsafeAccess() {}
 
   private static class UnsafeHolder {
-    public static final Unsafe UNSAFE;
+    private static final Unsafe UNSAFE;
 
     static {
       UNSAFE = getUnsafe();

exporters/common/src/testGrpcSenderProvider/java/io/opentelemetry/exporter/internal/grpc/GrpcExporterTest.java

@@ -121,7 +121,7 @@ void build_multipleSendersNoMatch() {
   private static class DummyServiceFutureStub
       extends MarshalerServiceStub<DummyMarshaler, Object, DummyServiceFutureStub> {
 
-    protected DummyServiceFutureStub(Channel channel, CallOptions callOptions) {
+    private DummyServiceFutureStub(Channel channel, CallOptions callOptions) {
       super(channel, callOptions);
     }
 
@@ -147,7 +147,7 @@ private static DummyServiceFutureStub newFutureStub(
 
   private static class DummyMarshaler extends MarshalerWithSize {
 
-    protected DummyMarshaler() {
+    private DummyMarshaler() {
       super(0);
     }
 

exporters/otlp/common/src/jmh/java/io/opentelemetry/exporter/internal/otlp/StringMarshalBenchmark.java

@@ -110,12 +110,12 @@ private static class TestMarshaler extends Marshaler {
       context = new MarshalerContext(/* marshalStringNoAllocation= */ true, useUnsafe);
     }
 
-    public void initialize(String string) {
+    private void initialize(String string) {
       value = string;
       size = StringAnyValueStatelessMarshaler.INSTANCE.getBinarySerializedSize(string, context);
     }
 
-    public void reset() {
+    private void reset() {
       context.reset();
     }
 

exporters/otlp/common/src/test/java/io/opentelemetry/exporter/internal/otlp/metrics/LowAllocationMetricsRequestMarshalerTest.java

@@ -128,7 +128,7 @@ void validateExemplar(ExemplarData exemplar) throws Exception {
       MarshalerContext context = new MarshalerContext();
       class TestMarshaler extends MarshalerWithSize {
 
-        protected TestMarshaler() {
+        TestMarshaler() {
           super(ExemplarStatelessMarshaler.INSTANCE.getBinarySerializedSize(exemplar, context));
         }
 
@@ -171,7 +171,7 @@ void validateSummary() throws Exception {
       MarshalerContext context = new MarshalerContext();
       class TestMarshaler extends MarshalerWithSize {
 
-        protected TestMarshaler() {
+        TestMarshaler() {
           super(SummaryStatelessMarshaler.INSTANCE.getBinarySerializedSize(summary, context));
         }
 

sdk-extensions/incubator/src/main/java/io/opentelemetry/sdk/extension/incubator/fileconfig/DeclarativeConfiguration.java

@@ -246,7 +246,7 @@ private static final class EnvLoad extends Load {
     private final LoadSettings settings;
     private final Map<String, String> environmentVariables;
 
-    public EnvLoad(LoadSettings settings, Map<String, String> environmentVariables) {
+    private EnvLoad(LoadSettings settings, Map<String, String> environmentVariables) {
       super(settings);
       this.settings = settings;
       this.environmentVariables = environmentVariables;
@@ -295,7 +295,7 @@ private static final class EnvComposer extends Composer {
     private static final int ESCAPE_SEQUENCE_LENGTH = ESCAPE_SEQUENCE.length();
     private static final char ESCAPE_SEQUENCE_REPLACEMENT = '$';
 
-    public EnvComposer(
+    private EnvComposer(
         LoadSettings settings, ParserImpl parser, Map<String, String> environmentVariables) {
       super(settings, parser);
       this.load = new Load(settings);

sdk/metrics/src/jmh/java/io/opentelemetry/sdk/metrics/TestSdk.java

@@ -82,7 +82,7 @@ public Tracer getTracer() {
   private abstract static class SdkBuilder {
     abstract Meter build();
 
-    protected static Tracer buildTracer() {
+    private static Tracer buildTracer() {
       return SdkTracerProvider.builder()
           .setSampler(Sampler.alwaysOn())
           .build()

sdk/metrics/src/jmh/java/io/opentelemetry/sdk/metrics/internal/aggregator/HistogramValueGenerator.java

@@ -49,7 +49,7 @@ private static class PoolSupplier implements DoubleSupplier {
     private final double[] pool;
     private final AtomicInteger idx = new AtomicInteger(0);
 
-    public PoolSupplier(double[] pool) {
+    private PoolSupplier(double[] pool) {
       this.pool = pool;
     }
 

sdk/trace/src/test/java/io/opentelemetry/sdk/trace/SdkTracerTest.java

@@ -151,7 +151,7 @@ public boolean isEndRequired() {
   private static class SimpleSpanOperation implements OperationUpdater {
     private final SdkTracer tracer;
 
-    public SimpleSpanOperation(SdkTracer tracer) {
+    private SimpleSpanOperation(SdkTracer tracer) {
       this.tracer = tracer;
     }
 
@@ -168,7 +168,7 @@ public void update() {
 
   private static class CountingSpanExporter implements SpanExporter {
 
-    public final AtomicLong numberOfSpansExported = new AtomicLong();
+    private final AtomicLong numberOfSpansExported = new AtomicLong();
 
     @Override
     public CompletableResultCode export(Collection<SpanData> spans) {