From da0841c095b3e41a1564b3931f056f7d10fca6bd Mon Sep 17 00:00:00 2001
From: Michael Bayne <mdb@threerings.net>
Date: Thu, 7 Feb 2008 19:59:54 +0000
Subject: [PATCH] None of these constructors actually throw a
 PersistenceException, so we'll go ahead and not declare that they do.

---
 .../com/samskivert/jdbc/depot/clause/FieldDefinition.java     | 4 ----
 src/java/com/samskivert/jdbc/depot/clause/FieldOverride.java  | 4 ----
 src/java/com/samskivert/jdbc/depot/clause/FromOverride.java   | 4 ----
 src/java/com/samskivert/jdbc/depot/clause/Join.java           | 3 ---
 4 files changed, 15 deletions(-)

diff --git a/src/java/com/samskivert/jdbc/depot/clause/FieldDefinition.java b/src/java/com/samskivert/jdbc/depot/clause/FieldDefinition.java
index f757874..b837a82 100644
--- a/src/java/com/samskivert/jdbc/depot/clause/FieldDefinition.java
+++ b/src/java/com/samskivert/jdbc/depot/clause/FieldDefinition.java
@@ -22,7 +22,6 @@ package com.samskivert.jdbc.depot.clause;
 
 import java.util.Collection;
 
-import com.samskivert.io.PersistenceException;
 import com.samskivert.jdbc.depot.PersistentRecord;
 import com.samskivert.jdbc.depot.expression.ColumnExp;
 import com.samskivert.jdbc.depot.expression.ExpressionVisitor;
@@ -41,19 +40,16 @@ import com.samskivert.jdbc.depot.expression.SQLExpression;
 public class FieldDefinition extends QueryClause
 {
     public FieldDefinition (String field, String str)
-        throws PersistenceException
     {
         this(field, new LiteralExp(str));
     }
 
     public FieldDefinition (String field, Class<? extends PersistentRecord> pClass, String pCol)
-        throws PersistenceException
     {
         this(field, new ColumnExp(pClass, pCol));
     }
 
     public FieldDefinition (String field, SQLExpression override)
-        throws PersistenceException
     {
         _field = field;
         _definition = override;
diff --git a/src/java/com/samskivert/jdbc/depot/clause/FieldOverride.java b/src/java/com/samskivert/jdbc/depot/clause/FieldOverride.java
index 94811c3..cf59a99 100644
--- a/src/java/com/samskivert/jdbc/depot/clause/FieldOverride.java
+++ b/src/java/com/samskivert/jdbc/depot/clause/FieldOverride.java
@@ -20,7 +20,6 @@
 
 package com.samskivert.jdbc.depot.clause;
 
-import com.samskivert.io.PersistenceException;
 import com.samskivert.jdbc.depot.PersistentRecord;
 import com.samskivert.jdbc.depot.expression.ColumnExp;
 import com.samskivert.jdbc.depot.expression.LiteralExp;
@@ -37,19 +36,16 @@ import com.samskivert.jdbc.depot.expression.SQLExpression;
 public class FieldOverride extends FieldDefinition
 {
     public FieldOverride (String field, String str)
-        throws PersistenceException
     {
         super(field, str);
     }
 
     public FieldOverride (String field, Class<? extends PersistentRecord> pClass, String pCol)
-        throws PersistenceException
     {
         super(field, pClass, pCol);
     }
 
     public FieldOverride (String field, SQLExpression override)
-        throws PersistenceException
     {
         super(field, override);
     }
diff --git a/src/java/com/samskivert/jdbc/depot/clause/FromOverride.java b/src/java/com/samskivert/jdbc/depot/clause/FromOverride.java
index 0ad1b42..4263069 100644
--- a/src/java/com/samskivert/jdbc/depot/clause/FromOverride.java
+++ b/src/java/com/samskivert/jdbc/depot/clause/FromOverride.java
@@ -24,7 +24,6 @@ import java.util.ArrayList;
 import java.util.Collection;
 import java.util.List;
 
-import com.samskivert.io.PersistenceException;
 import com.samskivert.jdbc.depot.PersistentRecord;
 import com.samskivert.jdbc.depot.expression.ExpressionVisitor;
 
@@ -34,21 +33,18 @@ import com.samskivert.jdbc.depot.expression.ExpressionVisitor;
 public class FromOverride extends QueryClause
 {
     public FromOverride (Class<? extends PersistentRecord> fromClass)
-        throws PersistenceException
     {
         _fromClasses.add(fromClass);
     }
 
     public FromOverride (Class<? extends PersistentRecord> fromClass1,
                          Class<? extends PersistentRecord> fromClass2)
-        throws PersistenceException
     {
         _fromClasses.add(fromClass1);
         _fromClasses.add(fromClass2);
     }
 
     public FromOverride (Collection<Class<? extends PersistentRecord>> fromClasses)
-        throws PersistenceException
     {
         _fromClasses.addAll(fromClasses);
     }
diff --git a/src/java/com/samskivert/jdbc/depot/clause/Join.java b/src/java/com/samskivert/jdbc/depot/clause/Join.java
index 3438416..4ee9ee7 100644
--- a/src/java/com/samskivert/jdbc/depot/clause/Join.java
+++ b/src/java/com/samskivert/jdbc/depot/clause/Join.java
@@ -22,7 +22,6 @@ package com.samskivert.jdbc.depot.clause;
 
 import java.util.Collection;
 
-import com.samskivert.io.PersistenceException;
 import com.samskivert.jdbc.depot.PersistentRecord;
 import com.samskivert.jdbc.depot.expression.ColumnExp;
 import com.samskivert.jdbc.depot.expression.ExpressionVisitor;
@@ -39,14 +38,12 @@ public class Join extends QueryClause
 
     public Join (Class<? extends PersistentRecord> pClass, String pCol,
                  Class<? extends PersistentRecord> joinClass, String jCol)
-        throws PersistenceException
     {
         _joinClass = joinClass;
         _joinCondition = new Equals(new ColumnExp(joinClass, jCol), new ColumnExp(pClass, pCol));
     }
 
     public Join (ColumnExp primary, ColumnExp join)
-        throws PersistenceException
     {
         _joinClass = join.getPersistentClass();
         _joinCondition = new Equals(primary, join);