diff options
| author | Brian S. O'Neill <bronee@gmail.com> | 2011-02-03 00:59:11 +0000 |
|---|---|---|
| committer | Brian S. O'Neill <bronee@gmail.com> | 2011-02-03 00:59:11 +0000 |
| commit | b130cf0025af0070c38d36e6d4b6902188ade9a0 (patch) | |
| tree | 04b6b109ac3635b1bae8d69d173950ae3c109fd2 /src/main/java/com/amazon/carbonado | |
| parent | 6722a5988985ce74dc0cdf8f1b0bef5b40c2fa1b (diff) | |
Support copying when property read or write methods are missing.
Diffstat (limited to 'src/main/java/com/amazon/carbonado')
| -rw-r--r-- | src/main/java/com/amazon/carbonado/gen/StorableCopier.java | 126 |
1 files changed, 81 insertions, 45 deletions
diff --git a/src/main/java/com/amazon/carbonado/gen/StorableCopier.java b/src/main/java/com/amazon/carbonado/gen/StorableCopier.java index f03790d..d358bae 100644 --- a/src/main/java/com/amazon/carbonado/gen/StorableCopier.java +++ b/src/main/java/com/amazon/carbonado/gen/StorableCopier.java @@ -18,6 +18,8 @@ package com.amazon.carbonado.gen;
+import java.lang.reflect.Method;
+import java.lang.reflect.Modifier;
import java.lang.reflect.Constructor;
import java.util.WeakHashMap;
@@ -242,18 +244,30 @@ public abstract class StorableCopier<S extends Storable, T extends Storable> { if (wrapperPropType.equals(delegatePropType)) {
// No conversion required.
- b = new CodeBuilder(mClassFile.addMethod(wrapperProp.getReadMethod()));
- b.loadThis();
- b.loadField("delegate", delegateType);
- b.invoke(delegateProp.getReadMethod());
- b.returnValue(wrapperPropType);
-
- b = new CodeBuilder(mClassFile.addMethod(wrapperProp.getWriteMethod()));
- b.loadThis();
- b.loadField("delegate", delegateType);
- b.loadLocal(b.getParameter(0));
- b.invoke(delegateProp.getWriteMethod());
- b.returnVoid();
+ Method m = canDefine(wrapperProp.getReadMethod());
+ if (m != null) {
+ b = new CodeBuilder(mClassFile.addMethod(m));
+ if (delegateProp.getReadMethod() == null) {
+ CodeBuilderUtil.blankValue(b, wrapperPropType);
+ } else {
+ b.loadThis();
+ b.loadField("delegate", delegateType);
+ b.invoke(delegateProp.getReadMethod());
+ }
+ b.returnValue(wrapperPropType);
+ }
+
+ m = canDefine(wrapperProp.getWriteMethod());
+ if (m != null) {
+ b = new CodeBuilder(mClassFile.addMethod(m));
+ if (delegateProp.getWriteMethod() != null) {
+ b.loadThis();
+ b.loadField("delegate", delegateType);
+ b.loadLocal(b.getParameter(0));
+ b.invoke(delegateProp.getWriteMethod());
+ }
+ b.returnVoid();
+ }
continue;
}
@@ -268,39 +282,51 @@ public abstract class StorableCopier<S extends Storable, T extends Storable> { // Convert primitive or boxed type.
- b = new CodeBuilder(mClassFile.addMethod(wrapperProp.getReadMethod()));
- b.loadThis();
- b.loadField("delegate", delegateType);
- b.invoke(delegateProp.getReadMethod());
- if (wrapperPropType.isPrimitive() && !delegatePropType.isPrimitive()) {
- // Check for null.
- b.dup();
- Label notNull = b.createLabel();
- b.ifNullBranch(notNull, false);
- CodeBuilderUtil.blankValue(b, wrapperPropType);
+ Method m = canDefine(wrapperProp.getReadMethod());
+ if (m != null) {
+ b = new CodeBuilder(mClassFile.addMethod(m));
+ if (delegateProp.getReadMethod() == null) {
+ CodeBuilderUtil.blankValue(b, wrapperPropType);
+ } else {
+ b.loadThis();
+ b.loadField("delegate", delegateType);
+ b.invoke(delegateProp.getReadMethod());
+ if (wrapperPropType.isPrimitive() && !delegatePropType.isPrimitive()) {
+ // Check for null.
+ b.dup();
+ Label notNull = b.createLabel();
+ b.ifNullBranch(notNull, false);
+ CodeBuilderUtil.blankValue(b, wrapperPropType);
+ b.returnValue(wrapperPropType);
+ notNull.setLocation();
+ }
+ b.convert(delegatePropType, wrapperPropType);
+ }
b.returnValue(wrapperPropType);
- notNull.setLocation();
}
- b.convert(delegatePropType, wrapperPropType);
- b.returnValue(wrapperPropType);
- b = new CodeBuilder(mClassFile.addMethod(wrapperProp.getWriteMethod()));
- b.loadThis();
- b.loadField("delegate", delegateType);
- b.loadLocal(b.getParameter(0));
- if (!wrapperPropType.isPrimitive() && delegatePropType.isPrimitive()) {
- // Check for null.
- Label notNull = b.createLabel();
- b.ifNullBranch(notNull, false);
- CodeBuilderUtil.blankValue(b, delegatePropType);
- b.invoke(delegateProp.getWriteMethod());
+ m = canDefine(wrapperProp.getWriteMethod());
+ if (m != null) {
+ b = new CodeBuilder(mClassFile.addMethod(m));
+ if (delegateProp.getWriteMethod() != null) {
+ b.loadThis();
+ b.loadField("delegate", delegateType);
+ b.loadLocal(b.getParameter(0));
+ if (!wrapperPropType.isPrimitive() && delegatePropType.isPrimitive()) {
+ // Check for null.
+ Label notNull = b.createLabel();
+ b.ifNullBranch(notNull, false);
+ CodeBuilderUtil.blankValue(b, delegatePropType);
+ b.invoke(delegateProp.getWriteMethod());
+ b.returnVoid();
+ notNull.setLocation();
+ b.loadLocal(b.getParameter(0));
+ }
+ b.convert(wrapperPropType, delegatePropType);
+ b.invoke(delegateProp.getWriteMethod());
+ }
b.returnVoid();
- notNull.setLocation();
- b.loadLocal(b.getParameter(0));
}
- b.convert(wrapperPropType, delegatePropType);
- b.invoke(delegateProp.getWriteMethod());
- b.returnVoid();
}
try {
@@ -314,12 +340,22 @@ public abstract class StorableCopier<S extends Storable, T extends Storable> { private void addUnmatchedProperty(StorableProperty<W> wrapperProp,
TypeDesc wrapperPropType)
{
- CodeBuilder b = new CodeBuilder(mClassFile.addMethod(wrapperProp.getReadMethod()));
- CodeBuilderUtil.blankValue(b, wrapperPropType);
- b.returnValue(wrapperPropType);
+ Method m = canDefine(wrapperProp.getReadMethod());
+ if (m != null) {
+ CodeBuilder b = new CodeBuilder(mClassFile.addMethod(m));
+ CodeBuilderUtil.blankValue(b, wrapperPropType);
+ b.returnValue(wrapperPropType);
+ }
- b = new CodeBuilder(mClassFile.addMethod(wrapperProp.getWriteMethod()));
- b.returnVoid();
+ m = canDefine(wrapperProp.getWriteMethod());
+ if (m != null) {
+ CodeBuilder b = new CodeBuilder(mClassFile.addMethod(m));
+ b.returnVoid();
+ }
+ }
+
+ private static Method canDefine(Method m) {
+ return (m == null || Modifier.isFinal(m.getModifiers())) ? null : m;
}
}
}
|