Skip to content

Commit 39db1ba

Browse files
committed
[java] JSpecify annotations for org.openqa.selenium.grid.jmx.JMXHelper
1 parent ef9b910 commit 39db1ba

File tree

2 files changed

+8
-2
lines changed

2 files changed

+8
-2
lines changed

java/src/org/openqa/selenium/grid/jmx/BUILD.bazel

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
load("//java:defs.bzl", "java_library")
1+
load("//java:defs.bzl", "artifact", "java_library")
22

33
java_library(
44
name = "jmx",
@@ -8,4 +8,7 @@ java_library(
88
"//java/src/org/openqa/selenium/grid/session:__pkg__",
99
"//java/test/org/openqa/selenium/grid:__subpackages__",
1010
],
11+
deps = [
12+
artifact("org.jspecify:jspecify"),
13+
],
1114
)

java/src/org/openqa/selenium/grid/jmx/JMXHelper.java

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,12 +22,15 @@
2222
import javax.management.InstanceAlreadyExistsException;
2323
import javax.management.MBeanServer;
2424
import javax.management.ObjectName;
25+
import org.jspecify.annotations.NullMarked;
26+
import org.jspecify.annotations.Nullable;
2527

28+
@NullMarked
2629
public class JMXHelper {
2730

2831
private static final Logger LOG = Logger.getLogger(JMXHelper.class.getName());
2932

30-
public MBean register(Object bean) {
33+
public @Nullable MBean register(Object bean) {
3134
MBeanServer mbs = ManagementFactory.getPlatformMBeanServer();
3235
MBean mBean = new MBean(bean);
3336
try {

0 commit comments

Comments
 (0)