You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@groovy.apache.org by su...@apache.org on 2020/07/13 23:54:20 UTC
[groovy] branch GROOVY_3_0_X updated: Revert the trivial tweaks
This is an automated email from the ASF dual-hosted git repository.
sunlan pushed a commit to branch GROOVY_3_0_X
in repository https://gitbox.apache.org/repos/asf/groovy.git
The following commit(s) were added to refs/heads/GROOVY_3_0_X by this push:
new 3941f78 Revert the trivial tweaks
3941f78 is described below
commit 3941f78890e9de8ed84730a60bdaf035f76fcefa
Author: Daniel Sun <su...@apache.org>
AuthorDate: Tue Jul 14 07:50:58 2020 +0800
Revert the trivial tweaks
All generated methods in the script of GROOVY-9601 all startsWith "my", so it is not common in the real world
(cherry picked from commit b9a50cf99fa94a330ef5a2ceac2d6c023a3ea0c4)
---
.../apache/groovy/ast/tools/MethodNodeUtils.java | 43 ++++++++++------------
1 file changed, 19 insertions(+), 24 deletions(-)
diff --git a/src/main/java/org/apache/groovy/ast/tools/MethodNodeUtils.java b/src/main/java/org/apache/groovy/ast/tools/MethodNodeUtils.java
index c1b1e34..190462c 100644
--- a/src/main/java/org/apache/groovy/ast/tools/MethodNodeUtils.java
+++ b/src/main/java/org/apache/groovy/ast/tools/MethodNodeUtils.java
@@ -81,32 +81,27 @@ public class MethodNodeUtils {
* @return the property name without the get/set/is prefix if a property or null
*/
public static String getPropertyName(final MethodNode mNode) {
- final int parameterCnt = mNode.getParameters().length;
-
- if (parameterCnt < 2) {
- final String name = mNode.getName();
- final int nameLength = name.length();
- if (nameLength > 2) {
- switch (name.charAt(0)) {
- case 'g':
- if (nameLength > 3 && parameterCnt == 0 && name.charAt(1) == 'e' && name.charAt(2) == 't' && !mNode.getReturnType().equals(ClassHelper.VOID_TYPE)) {
- return decapitalize(name.substring(3));
- }
- break;
- case 's':
- if (nameLength > 3 && parameterCnt == 1 && name.charAt(1) == 'e' && name.charAt(2) == 't' /*&& mNode.getReturnType().equals(ClassHelper.VOID_TYPE)*/) {
- return decapitalize(name.substring(3));
- }
- break;
- case 'i':
- if (parameterCnt == 0 && name.charAt(1) == 's' && (mNode.getReturnType().equals(ClassHelper.boolean_TYPE) /*|| mNode.getReturnType().equals(ClassHelper.Boolean_TYPE)*/)) {
- return decapitalize(name.substring(2));
- }
- break;
- }
+ final String name = mNode.getName();
+ final int nameLength = name.length();
+ if (nameLength > 2) {
+ switch (name.charAt(0)) {
+ case 'g':
+ if (nameLength > 3 && name.charAt(1) == 'e' && name.charAt(2) == 't' && mNode.getParameters().length == 0 && !mNode.getReturnType().equals(ClassHelper.VOID_TYPE)) {
+ return decapitalize(name.substring(3));
+ }
+ break;
+ case 's':
+ if (nameLength > 3 && name.charAt(1) == 'e' && name.charAt(2) == 't' && mNode.getParameters().length == 1 /*&& mNode.getReturnType().equals(ClassHelper.VOID_TYPE)*/) {
+ return decapitalize(name.substring(3));
+ }
+ break;
+ case 'i':
+ if (name.charAt(1) == 's' && mNode.getParameters().length == 0 && (mNode.getReturnType().equals(ClassHelper.boolean_TYPE) /*|| mNode.getReturnType().equals(ClassHelper.Boolean_TYPE)*/)) {
+ return decapitalize(name.substring(2));
+ }
+ break;
}
}
-
return null;
}