Skip to content

Commit c307736

Browse files
authored
Merge pull request #1817 from hazendaz/master
[ci] Cleanup a couple pmd issues
2 parents 69ad93f + 4c9b41a commit c307736

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

src/main/java/org/apache/ibatis/binding/MapperProxy.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/**
2-
* Copyright 2009-2019 the original author or authors.
2+
* Copyright 2009-2020 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.
@@ -82,14 +82,14 @@ public Object invoke(Object proxy, Method method, Object[] args) throws Throwabl
8282
if (Object.class.equals(method.getDeclaringClass())) {
8383
return method.invoke(this, args);
8484
} else {
85-
return cachedInvoker(proxy, method, args).invoke(proxy, method, args, sqlSession);
85+
return cachedInvoker(method).invoke(proxy, method, args, sqlSession);
8686
}
8787
} catch (Throwable t) {
8888
throw ExceptionUtil.unwrapThrowable(t);
8989
}
9090
}
9191

92-
private MapperMethodInvoker cachedInvoker(Object proxy, Method method, Object[] args) throws Throwable {
92+
private MapperMethodInvoker cachedInvoker(Method method) throws Throwable {
9393
try {
9494
return methodCache.computeIfAbsent(method, m -> {
9595
if (m.isDefault()) {

src/main/java/org/apache/ibatis/executor/keygen/Jdbc3KeyGenerator.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/**
2-
* Copyright 2009-2019 the original author or authors.
2+
* Copyright 2009-2020 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.
@@ -98,7 +98,7 @@ private void assignKeys(Configuration configuration, ResultSet rs, ResultSetMeta
9898
} else if (parameter instanceof ArrayList && !((ArrayList<?>) parameter).isEmpty()
9999
&& ((ArrayList<?>) parameter).get(0) instanceof ParamMap) {
100100
// Multi-param or single param with @Param in batch operation
101-
assignKeysToParamMapList(configuration, rs, rsmd, keyProperties, ((ArrayList<ParamMap<?>>) parameter));
101+
assignKeysToParamMapList(configuration, rs, rsmd, keyProperties, (ArrayList<ParamMap<?>>) parameter);
102102
} else {
103103
// Single param without @Param
104104
assignKeysToParam(configuration, rs, rsmd, keyProperties, parameter);

0 commit comments

Comments
 (0)