From f07694895ef4591fbc204626c0fede5b324d0e11 Mon Sep 17 00:00:00 2001 From: Michael Bayne Date: Mon, 4 Aug 2008 00:49:37 +0000 Subject: [PATCH] Fixed some bogus null checks and assignments. --- src/java/com/samskivert/jdbc/depot/BuildVisitor.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/java/com/samskivert/jdbc/depot/BuildVisitor.java b/src/java/com/samskivert/jdbc/depot/BuildVisitor.java index 7816c9b..10ce038 100644 --- a/src/java/com/samskivert/jdbc/depot/BuildVisitor.java +++ b/src/java/com/samskivert/jdbc/depot/BuildVisitor.java @@ -515,12 +515,12 @@ public abstract class BuildVisitor implements ExpressionVisitor throws Exception { DepotMarshaller dm = _types.getMarshaller(type); - FieldMarshaller fm = dm.getFieldMarshaller(field); if (dm == null) { throw new IllegalArgumentException( "Unknown field on persistent record [record=" + type + ", field=" + field + "]"); } + FieldMarshaller fm = dm.getFieldMarshaller(field); appendIdentifier(fm.getColumnName()); } @@ -530,12 +530,12 @@ public abstract class BuildVisitor implements ExpressionVisitor throws Exception { DepotMarshaller dm = _types.getMarshaller(type); - FieldMarshaller fm = dm.getFieldMarshaller(field); if (dm == null) { throw new IllegalArgumentException( "Unknown field on persistent record [record=" + type + ", field=" + field + "]"); } + FieldMarshaller fm = dm.getFieldMarshaller(field); Map fieldOverrides = _definitions.get(type); if (fieldOverrides != null) { // first, see if there's a field override