Skip to content

Commit 4f4dbfa

Browse files
scordioscordio
committed
Introduce ErrorProne, fix compiler warnings
Signed-off-by: Stefano Cordio <[email protected]>
1 parent 08c4cb1 commit 4f4dbfa

File tree

197 files changed

+716
-711
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

197 files changed

+716
-711
lines changed

‎.mvn/jvm.config

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
--add-exports jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED
2+
--add-exports jdk.compiler/com.sun.tools.javac.file=ALL-UNNAMED
3+
--add-exports jdk.compiler/com.sun.tools.javac.main=ALL-UNNAMED
4+
--add-exports jdk.compiler/com.sun.tools.javac.model=ALL-UNNAMED
5+
--add-exports jdk.compiler/com.sun.tools.javac.parser=ALL-UNNAMED
6+
--add-exports jdk.compiler/com.sun.tools.javac.processing=ALL-UNNAMED
7+
--add-exports jdk.compiler/com.sun.tools.javac.tree=ALL-UNNAMED
8+
--add-exports jdk.compiler/com.sun.tools.javac.util=ALL-UNNAMED
9+
--add-opens jdk.compiler/com.sun.tools.javac.code=ALL-UNNAMED
10+
--add-opens jdk.compiler/com.sun.tools.javac.comp=ALL-UNNAMED

‎pom.xml

Lines changed: 54 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -181,7 +181,61 @@
181181
<release>${java.version}</release>
182182
<compilerArgs>
183183
<compilerArg>-parameters</compilerArg>
184+
<!-- https://errorprone.info/docs/installation#maven -->
185+
<compilerArg>-XDcompilePolicy=simple</compilerArg>
186+
<compilerArg>--should-stop=ifError=FLOW</compilerArg>
187+
<compilerArg>
188+
-Xplugin:ErrorProne
189+
-Xep:AlmostJavadoc:OFF
190+
-Xep:BadInstanceof:OFF <!-- FIXME gh-4839 -->
191+
-Xep:ByteBufferBackingArray:OFF
192+
-Xep:ClassCanBeStatic:OFF
193+
-Xep:CollectionUndefinedEquality:OFF
194+
-Xep:DefaultCharset:OFF
195+
-Xep:DirectInvocationOnMock:OFF
196+
-Xep:DoNotCallSuggester:OFF
197+
-Xep:EmptyCatch:OFF
198+
-Xep:EqualsGetClass:OFF
199+
-Xep:Finally:OFF
200+
-Xep:FutureReturnValueIgnored:OFF
201+
-Xep:HidingField:OFF
202+
-Xep:ImmutableEnumChecker:OFF
203+
-Xep:InlineMeSuggester:OFF
204+
-Xep:InputStreamSlowMultibyteRead:OFF
205+
-Xep:JavaTimeDefaultTimeZone:OFF
206+
-Xep:JavaUtilDate:OFF
207+
-Xep:JdkObsolete:OFF
208+
-Xep:MissingSummary:OFF
209+
-Xep:MixedMutabilityReturnType:OFF
210+
-Xep:MutablePublicArray:OFF
211+
-Xep:NonAtomicVolatileUpdate:OFF
212+
-Xep:RedundantControlFlow:OFF
213+
-Xep:ReferenceEquality:OFF
214+
-Xep:StaticAssignmentInConstructor:OFF
215+
-Xep:StaticAssignmentOfThrowable:OFF
216+
-Xep:StaticMockMember:OFF
217+
-Xep:StreamResourceLeak:OFF
218+
-Xep:StringCaseLocaleUsage:OFF
219+
-Xep:StringSplitter:OFF
220+
-Xep:SynchronizeOnNonFinalField:OFF
221+
-Xep:ThreadLocalUsage:OFF
222+
-Xep:ThreadPriorityCheck:OFF
223+
-Xep:TypeParameterUnusedInFormals:OFF
224+
-Xep:UndefinedEquals:OFF
225+
-Xep:UnnecessaryStringBuilder:OFF
226+
-Xep:UnusedMethod:OFF
227+
-Xep:UnusedVariable:OFF
228+
-Xep:WaitNotInLoop:OFF
229+
</compilerArg>
184230
</compilerArgs>
231+
<failOnWarning>true</failOnWarning>
232+
<annotationProcessorPaths>
233+
<path>
234+
<groupId>com.google.errorprone</groupId>
235+
<artifactId>error_prone_core</artifactId>
236+
<version>2.38.0</version>
237+
</path>
238+
</annotationProcessorPaths>
185239
</configuration>
186240
</plugin>
187241
<plugin>

‎spring-batch-core/src/main/java/org/springframework/batch/core/Entity.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -146,7 +146,7 @@ public boolean equals(Object other) {
146146
@Override
147147
public int hashCode() {
148148
if (id == null) {
149-
return super.hashCode();
149+
return System.identityHashCode(this);
150150
}
151151
return 39 + 87 * id.hashCode();
152152
}

‎spring-batch-core/src/main/java/org/springframework/batch/core/JobExecution.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@
1818

1919
import java.io.IOException;
2020
import java.io.ObjectInputStream;
21+
import java.time.Clock;
2122
import java.time.LocalDateTime;
2223
import java.util.ArrayList;
2324
import java.util.Collection;
@@ -203,7 +204,7 @@ public void upgradeStatus(BatchStatus status) {
203204
/**
204205
* Convenience getter for the {@code id} of the enclosing job. Useful for DAO
205206
* implementations.
206-
* @return the @{code id} of the enclosing job.
207+
* @return the {@code id} of the enclosing job.
207208
*/
208209
public Long getJobId() {
209210
if (jobInstance != null) {

‎spring-batch-core/src/main/java/org/springframework/batch/core/JobParameters.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -373,7 +373,7 @@ public String toString() {
373373
for (Map.Entry<String, JobParameter<?>> entry : this.parameters.entrySet()) {
374374
parameters.add(String.format("'%s':'%s'", entry.getKey(), entry.getValue()));
375375
}
376-
return new StringBuilder("{").append(String.join(",", parameters)).append("}").toString();
376+
return "{" + String.join(",", parameters) + "}";
377377
}
378378

379379
}

‎spring-batch-core/src/main/java/org/springframework/batch/core/StepExecution.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -492,7 +492,7 @@ public boolean equals(Object obj) {
492492
return super.equals(obj);
493493
}
494494

495-
return stepName.equals(other.getStepName()) && (jobExecutionId.equals(other.getJobExecutionId()))
495+
return stepName.equals(other.getStepName()) && jobExecutionId.equals(other.getJobExecutionId())
496496
&& getId().equals(other.getId());
497497
}
498498

‎spring-batch-core/src/main/java/org/springframework/batch/core/aot/CoreRuntimeHints.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -130,7 +130,7 @@ public void registerHints(RuntimeHints hints, ClassLoader classLoader) {
130130
.proxiedInterfaces(SpringProxy.class, Advised.class, DecoratingProxy.class));
131131

132132
// reflection hints
133-
hints.reflection().registerType(Types.class, MemberCategory.DECLARED_FIELDS);
133+
hints.reflection().registerType(Types.class, MemberCategory.ACCESS_DECLARED_FIELDS);
134134
hints.reflection().registerType(JobContext.class, MemberCategory.INVOKE_PUBLIC_METHODS);
135135
hints.reflection().registerType(StepContext.class, MemberCategory.INVOKE_PUBLIC_METHODS);
136136
hints.reflection().registerType(JobParameter.class, MemberCategory.values());

‎spring-batch-core/src/main/java/org/springframework/batch/core/configuration/annotation/BatchObservabilityBeanPostProcessor.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -50,11 +50,11 @@ public Object postProcessAfterInitialization(Object bean, String beanName) throw
5050
try {
5151
if (bean instanceof AbstractJob || bean instanceof AbstractStep) {
5252
ObservationRegistry observationRegistry = this.beanFactory.getBean(ObservationRegistry.class);
53-
if (bean instanceof AbstractJob) {
54-
((AbstractJob) bean).setObservationRegistry(observationRegistry);
53+
if (bean instanceof AbstractJob job) {
54+
job.setObservationRegistry(observationRegistry);
5555
}
56-
if (bean instanceof AbstractStep) {
57-
((AbstractStep) bean).setObservationRegistry(observationRegistry);
56+
if (bean instanceof AbstractStep step) {
57+
step.setObservationRegistry(observationRegistry);
5858
}
5959
}
6060
}

‎spring-batch-core/src/main/java/org/springframework/batch/core/configuration/annotation/BatchRegistrar.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -50,8 +50,6 @@ class BatchRegistrar implements ImportBeanDefinitionRegistrar {
5050

5151
private static final Log LOGGER = LogFactory.getLog(BatchRegistrar.class);
5252

53-
private static final String MISSING_ANNOTATION_ERROR_MESSAGE = "EnableBatchProcessing is not present on importing class '%s' as expected";
54-
5553
private static final String JOB_REPOSITORY = "jobRepository";
5654

5755
private static final String JOB_OPERATOR = "jobOperator";
@@ -81,7 +79,8 @@ public void registerBeanDefinitions(AnnotationMetadata importingClassMetadata, B
8179
private void validateState(AnnotationMetadata importingClassMetadata) {
8280
if (!importingClassMetadata.isAnnotated(EnableBatchProcessing.class.getName())) {
8381
String className = importingClassMetadata.getClassName();
84-
String errorMessage = String.format(MISSING_ANNOTATION_ERROR_MESSAGE, className);
82+
String errorMessage = "EnableBatchProcessing is not present on importing class '%s' as expected"
83+
.formatted(className);
8584
throw new IllegalStateException(errorMessage);
8685
}
8786
}
@@ -255,6 +254,7 @@ private void registerJobOperator(BeanDefinitionRegistry registry, EnableBatchPro
255254
if (registry.containsBeanDefinition(taskExecutorRef)) {
256255
beanDefinitionBuilder.addPropertyReference("taskExecutor", taskExecutorRef);
257256
}
257+
@SuppressWarnings("removal")
258258
String jobParametersConverterRef = batchAnnotation.jobParametersConverterRef();
259259
if (registry.containsBeanDefinition(jobParametersConverterRef)) {
260260
beanDefinitionBuilder.addPropertyReference("jobParametersConverter", jobParametersConverterRef);

‎spring-batch-core/src/main/java/org/springframework/batch/core/configuration/support/AbstractApplicationContextFactory.java

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@
1919
import java.util.ArrayList;
2020
import java.util.Arrays;
2121
import java.util.Collection;
22+
import java.util.Collections;
2223
import java.util.List;
2324

2425
import org.apache.commons.logging.Log;
@@ -196,13 +197,11 @@ protected void prepareContext(ConfigurableApplicationContext parent, Configurabl
196197
protected void prepareBeanFactory(ConfigurableListableBeanFactory parent,
197198
ConfigurableListableBeanFactory beanFactory) {
198199
if (copyConfiguration && parent != null) {
199-
List<BeanPostProcessor> parentPostProcessors = new ArrayList<>();
200-
List<BeanPostProcessor> childPostProcessors = new ArrayList<>();
201-
202-
childPostProcessors.addAll(beanFactory instanceof AbstractBeanFactory
203-
? ((AbstractBeanFactory) beanFactory).getBeanPostProcessors() : new ArrayList<>());
204-
parentPostProcessors.addAll(parent instanceof AbstractBeanFactory
205-
? ((AbstractBeanFactory) parent).getBeanPostProcessors() : new ArrayList<>());
200+
List<BeanPostProcessor> childPostProcessors = new ArrayList<>(
201+
beanFactory instanceof AbstractBeanFactory factory ? factory.getBeanPostProcessors()
202+
: new ArrayList<>());
203+
List<BeanPostProcessor> parentPostProcessors = new ArrayList<>(parent instanceof AbstractBeanFactory factory
204+
? factory.getBeanPostProcessors() : new ArrayList<>());
206205

207206
try {
208207
Class<?> applicationContextAwareProcessorClass = ClassUtils.forName(
@@ -237,8 +236,8 @@ protected void prepareBeanFactory(ConfigurableListableBeanFactory parent,
237236

238237
beanFactory.copyConfigurationFrom(parent);
239238

240-
List<BeanPostProcessor> beanPostProcessors = beanFactory instanceof AbstractBeanFactory
241-
? ((AbstractBeanFactory) beanFactory).getBeanPostProcessors() : new ArrayList<>();
239+
List<BeanPostProcessor> beanPostProcessors = beanFactory instanceof AbstractBeanFactory abstractBeanFactory
240+
? abstractBeanFactory.getBeanPostProcessors() : new ArrayList<>();
242241

243242
beanPostProcessors.clear();
244243
beanPostProcessors.addAll(aggregatedPostProcessors);

‎spring-batch-core/src/main/java/org/springframework/batch/core/configuration/support/AutomaticJobRegistrar.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -79,8 +79,8 @@ public void setApplicationContext(ApplicationContext applicationContext) {
7979
* use
8080
*/
8181
public void addApplicationContextFactory(ApplicationContextFactory applicationContextFactory) {
82-
if (applicationContextFactory instanceof ApplicationContextAware) {
83-
((ApplicationContextAware) applicationContextFactory).setApplicationContext(applicationContext);
82+
if (applicationContextFactory instanceof ApplicationContextAware applicationContextAware) {
83+
applicationContextAware.setApplicationContext(applicationContext);
8484
}
8585
this.applicationContextFactories.add(applicationContextFactory);
8686
}

‎spring-batch-core/src/main/java/org/springframework/batch/core/configuration/support/DefaultJobLoader.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -255,11 +255,11 @@ private void doRegister(ConfigurableApplicationContext context, Job job) throws
255255
jobRegistry.register(jobFactory);
256256

257257
if (stepRegistry != null) {
258-
if (!(job instanceof StepLocator)) {
258+
if (!(job instanceof StepLocator stepLocator)) {
259259
throw new UnsupportedOperationException("Cannot locate steps from a Job that is not a StepLocator: job="
260260
+ job.getName() + " does not implement StepLocator");
261261
}
262-
stepRegistry.register(job.getName(), getSteps((StepLocator) job, context));
262+
stepRegistry.register(job.getName(), getSteps(stepLocator, context));
263263
}
264264
}
265265

‎spring-batch-core/src/main/java/org/springframework/batch/core/configuration/support/GenericApplicationContextFactory.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -126,7 +126,7 @@ protected void prepareBeanFactory(ConfigurableListableBeanFactory beanFactory) {
126126
GenericApplicationContextFactory.this.prepareBeanFactory(parentBeanFactory, beanFactory);
127127
for (Class<? extends BeanFactoryPostProcessor> cls : getBeanFactoryPostProcessorClasses()) {
128128
for (String name : parent.getBeanNamesForType(cls)) {
129-
beanFactory.registerSingleton(name, (parent.getBean(name)));
129+
beanFactory.registerSingleton(name, parent.getBean(name));
130130
}
131131
}
132132
}

‎spring-batch-core/src/main/java/org/springframework/batch/core/configuration/support/GroupAwareJob.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -99,8 +99,8 @@ public JobParametersValidator getJobParametersValidator() {
9999

100100
@Override
101101
public boolean equals(Object obj) {
102-
if (obj instanceof GroupAwareJob) {
103-
return ((GroupAwareJob) obj).delegate.equals(delegate);
102+
if (obj instanceof GroupAwareJob groupAwareJob) {
103+
return groupAwareJob.delegate.equals(delegate);
104104
}
105105
return false;
106106
}

‎spring-batch-core/src/main/java/org/springframework/batch/core/configuration/xml/AbstractFlowParser.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -404,7 +404,7 @@ protected static Collection<BeanDefinition> createTransition(FlowExecutionStatus
404404
endBuilder.addConstructorArgValue(exitCodeExists ? exitCode : status.getName());
405405

406406
String endName = (status == FlowExecutionStatus.STOPPED ? STOP_ELE
407-
: status == FlowExecutionStatus.FAILED ? FAIL_ELE : END_ELE) + (endCounter++);
407+
: status == FlowExecutionStatus.FAILED ? FAIL_ELE : END_ELE) + endCounter++;
408408
endBuilder.addConstructorArgValue(endName);
409409

410410
endBuilder.addConstructorArgValue(abandon);

‎spring-batch-core/src/main/java/org/springframework/batch/core/configuration/xml/AbstractStepParser.java

Lines changed: 6 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -118,16 +118,13 @@ protected AbstractBeanDefinition parseStep(Element stepElement, ParserContext pa
118118
new TaskletParser().parseTasklet(stepElement, nestedElement, bd, parserContext, stepUnderspecified);
119119
}
120120
else if (FLOW_ELE.equals(name)) {
121-
boolean stepUnderspecified = CoreNamespaceUtils.isUnderspecified(stepElement);
122-
parseFlow(stepElement, nestedElement, bd, parserContext, stepUnderspecified);
121+
parseFlow(stepElement, nestedElement, bd);
123122
}
124123
else if (PARTITION_ELE.equals(name)) {
125-
boolean stepUnderspecified = CoreNamespaceUtils.isUnderspecified(stepElement);
126-
parsePartition(stepElement, nestedElement, bd, parserContext, stepUnderspecified, jobFactoryRef);
124+
parsePartition(stepElement, nestedElement, bd, parserContext, jobFactoryRef);
127125
}
128126
else if (JOB_ELE.equals(name)) {
129-
boolean stepUnderspecified = CoreNamespaceUtils.isUnderspecified(stepElement);
130-
parseJob(stepElement, nestedElement, bd, parserContext, stepUnderspecified);
127+
parseJob(nestedElement, bd, parserContext);
131128
}
132129
else if ("description".equals(name)) {
133130
bd.setDescription(nestedElement.getTextContent());
@@ -199,7 +196,7 @@ else if (ns.equals("http://www.springframework.org/schema/batch")) {
199196
}
200197

201198
private void parsePartition(Element stepElement, Element partitionElement, AbstractBeanDefinition bd,
202-
ParserContext parserContext, boolean stepUnderspecified, String jobFactoryRef) {
199+
ParserContext parserContext, String jobFactoryRef) {
203200

204201
bd.setBeanClass(StepParserStepFactoryBean.class);
205202
bd.setAttribute("isNamespaceStep", true);
@@ -258,8 +255,7 @@ else if (inlineStepElement != null) {
258255

259256
}
260257

261-
private void parseJob(Element stepElement, Element jobElement, AbstractBeanDefinition bd,
262-
ParserContext parserContext, boolean stepUnderspecified) {
258+
private void parseJob(Element jobElement, AbstractBeanDefinition bd, ParserContext parserContext) {
263259

264260
bd.setBeanClass(StepParserStepFactoryBean.class);
265261
bd.setAttribute("isNamespaceStep", true);
@@ -285,8 +281,7 @@ private void parseJob(Element stepElement, Element jobElement, AbstractBeanDefin
285281

286282
}
287283

288-
private void parseFlow(Element stepElement, Element flowElement, AbstractBeanDefinition bd,
289-
ParserContext parserContext, boolean stepUnderspecified) {
284+
private void parseFlow(Element stepElement, Element flowElement, AbstractBeanDefinition bd) {
290285

291286
bd.setBeanClass(StepParserStepFactoryBean.class);
292287
bd.setAttribute("isNamespaceStep", true);

‎spring-batch-core/src/main/java/org/springframework/batch/core/configuration/xml/CoreNamespacePostProcessor.java

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -115,15 +115,13 @@ public Object postProcessBeforeInitialization(Object bean, String beanName) thro
115115
* @return the bean with default collaborators injected into it
116116
*/
117117
private Object injectDefaults(Object bean) {
118-
if (bean instanceof JobParserJobFactoryBean) {
119-
JobParserJobFactoryBean fb = (JobParserJobFactoryBean) bean;
118+
if (bean instanceof JobParserJobFactoryBean fb) {
120119
JobRepository jobRepository = fb.getJobRepository();
121120
if (jobRepository == null) {
122121
fb.setJobRepository((JobRepository) applicationContext.getBean(DEFAULT_JOB_REPOSITORY_NAME));
123122
}
124123
}
125-
else if (bean instanceof StepParserStepFactoryBean) {
126-
StepParserStepFactoryBean<?, ?> fb = (StepParserStepFactoryBean<?, ?>) bean;
124+
else if (bean instanceof StepParserStepFactoryBean<?, ?> fb) {
127125
JobRepository jobRepository = fb.getJobRepository();
128126
if (jobRepository == null) {
129127
fb.setJobRepository((JobRepository) applicationContext.getBean(DEFAULT_JOB_REPOSITORY_NAME));

‎spring-batch-core/src/main/java/org/springframework/batch/core/configuration/xml/ExceptionElementParser.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -32,8 +32,8 @@ public ManagedMap<TypedStringValue, Boolean> parse(Element element, ParserContex
3232
if (children.size() == 1) {
3333
ManagedMap<TypedStringValue, Boolean> map = new ManagedMap<>();
3434
Element exceptionClassesElement = children.get(0);
35-
addExceptionClasses("include", true, exceptionClassesElement, map, parserContext);
36-
addExceptionClasses("exclude", false, exceptionClassesElement, map, parserContext);
35+
addExceptionClasses("include", true, exceptionClassesElement, map);
36+
addExceptionClasses("exclude", false, exceptionClassesElement, map);
3737
map.put(new TypedStringValue(ForceRollbackForWriteSkipException.class.getName(), Class.class), true);
3838
return map;
3939
}
@@ -46,7 +46,7 @@ else if (children.size() > 1) {
4646
}
4747

4848
private void addExceptionClasses(String elementName, boolean include, Element exceptionClassesElement,
49-
ManagedMap<TypedStringValue, Boolean> map, ParserContext parserContext) {
49+
ManagedMap<TypedStringValue, Boolean> map) {
5050
for (Element child : DomUtils.getChildElementsByTagName(exceptionClassesElement, elementName)) {
5151
String className = child.getAttribute("class");
5252
map.put(new TypedStringValue(className, Class.class), include);

‎spring-batch-core/src/main/java/org/springframework/batch/core/configuration/xml/JobParser.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,7 @@ protected void doParse(Element element, ParserContext parserContext, BeanDefinit
103103
builder.addPropertyValue("restartable", restartableAttribute);
104104
}
105105

106-
String incrementer = (element.getAttribute("incrementer"));
106+
String incrementer = element.getAttribute("incrementer");
107107
if (StringUtils.hasText(incrementer)) {
108108
builder.addPropertyReference("jobParametersIncrementer", incrementer);
109109
}

‎spring-batch-core/src/main/java/org/springframework/batch/core/configuration/xml/SimpleFlowFactoryBean.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -205,7 +205,7 @@ public FlowExecutionStatus handle(FlowExecutor executor) throws Exception {
205205

206206
@Override
207207
public Collection<Flow> getFlows() {
208-
return (state instanceof FlowHolder) ? ((FlowHolder) state).getFlows() : Collections.<Flow>emptyList();
208+
return (state instanceof FlowHolder flowHolder) ? flowHolder.getFlows() : Collections.emptyList();
209209
}
210210

211211
}

0 commit comments

Comments
 (0)