From f54c74e385236c3f7e8756e2af8b7ea48c052549 Mon Sep 17 00:00:00 2001
From: Par Winzell <zell@threerings.net>
Date: Thu, 6 May 2010 14:37:18 +0000
Subject: [PATCH] Abstract records should never be directly referenced; fail as
 early as possible and explain what's going on.

---
 src/java/com/samskivert/depot/impl/DepotMarshaller.java | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/src/java/com/samskivert/depot/impl/DepotMarshaller.java b/src/java/com/samskivert/depot/impl/DepotMarshaller.java
index 2dbe7f4..01dc620 100644
--- a/src/java/com/samskivert/depot/impl/DepotMarshaller.java
+++ b/src/java/com/samskivert/depot/impl/DepotMarshaller.java
@@ -90,6 +90,9 @@ public class DepotMarshaller<T extends PersistentRecord>
     {
         _pClass = pClass;
 
+        Preconditions.checkArgument(!java.lang.reflect.Modifier.isAbstract(pClass.getModifiers()),
+            "Can't handle reference to abstract record: " + pClass.getName());
+
         Entity entity = pClass.getAnnotation(Entity.class);
 
         // see if this is a computed entity