Author: mprudhom Date: Wed Sep 27 15:50:08 2006 New Revision: 450619 URL: http://svn.apache.org/viewvc?view=rev&rev=450619 Log: Null checks.
Modified: incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/PCEnhancer.java Modified: incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/PCEnhancer.java URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/PCEnhancer.java?view=diff&rev=450619&r1=450618&r2=450619 ============================================================================== --- incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/PCEnhancer.java (original) +++ incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/enhance/PCEnhancer.java Wed Sep 27 15:50:08 2006 @@ -363,7 +363,7 @@ FieldMetaData[] fmds = _meta.getDeclaredFields(); Method meth; BCMethod getter, setter = null; - BCField returned, assigned; + BCField returned, assigned = null; for (int i = 0; i < fmds.length; i++) { if (!(fmds[i].getBackingMember() instanceof Method)) { addViolation("property-bad-member", @@ -408,11 +408,15 @@ } } - assigned = getAssignedField(setter); + if (setter != null) + assigned = getAssignedField(setter); + if (assigned != null) { if (_backingFields == null) _backingFields = new HashMap(); - _backingFields.put(setter.getName(), assigned.getName()); + + if (setter != null) + _backingFields.put(setter.getName(), assigned.getName()); if (assigned != returned) addViolation("property-setter-getter-mismatch",