Relaxing the throw specifiers for the destructors for Node, TypeNode, the context...
authorTim King <taking@google.com>
Thu, 1 Sep 2016 08:28:02 +0000 (01:28 -0700)
committerTim King <taking@google.com>
Thu, 1 Sep 2016 08:28:02 +0000 (01:28 -0700)
25 files changed:
src/context/cdchunk_list.h
src/context/cdhashmap.h
src/context/cdinsert_hashmap.h
src/context/cdlist.h
src/context/cdo.h
src/context/cdtrail_hashmap.h
src/context/context.cpp
src/context/context.h
src/context/context_mm.cpp
src/context/context_mm.h
src/context/stacking_vector.h
src/expr/node.h
src/expr/type_node.h
src/prop/bvminisat/bvminisat.cpp
src/prop/bvminisat/bvminisat.h
src/prop/cryptominisat.cpp
src/prop/cryptominisat.h
src/prop/minisat/minisat.cpp
src/prop/minisat/minisat.h
src/prop/sat_solver.h
src/theory/bv/bv_inequality_graph.h
src/theory/shared_terms_database.cpp
src/theory/shared_terms_database.h
src/theory/uf/equality_engine.cpp
src/theory/uf/equality_engine.h

index 3fa3c6b5d06050bd21340cbe514a30267bbba94b..e75f4de4d7b4bcec266e7f2893e4de83f3d724b0 100644 (file)
@@ -137,7 +137,7 @@ protected:
       d_size(size),
       d_sizeAlloc(sizeAlloc) {
     }
-    ~CDChunkListSave() throw() {
+    ~CDChunkListSave() {
       this->destroy();
     }
     ContextObj* save(ContextMemoryManager* pCMM) {
@@ -322,7 +322,7 @@ public:
   /**
    * Destructor: delete the list
    */
-  ~CDChunkList() throw(AssertionException) {
+  ~CDChunkList() {
     this->destroy();
 
     if(this->d_callDestructor) {
index 884234eb8c031788c8ed32aef0ed04fab193ed42..6ae74fbde94689d20e8e40c5133558c585c09406 100644 (file)
@@ -236,7 +236,7 @@ public:
     }
   }
 
-  ~CDOhash_map() throw(AssertionException) {
+  ~CDOhash_map() {
     destroy();
   }
 
@@ -327,7 +327,7 @@ public:
     d_trash() {
   }
 
-  ~CDHashMap() throw(AssertionException) {
+  ~CDHashMap() {
     Debug("gc") << "cdhashmap" << this
                 << " disappearing, destroying..." << std::endl;
     destroy();
index 6e772068bd5dac3490abb7acb73b9e428af7a12c..e901e9413e4368d7a296cb498dbd9715e6b35783 100644 (file)
@@ -271,7 +271,7 @@ public:
   /**
    * Destructor: delete the d_insertMap
    */
-  ~CDInsertHashMap() throw(AssertionException) {
+  ~CDInsertHashMap() {
     this->destroy();
     delete d_insertMap;
   }
index 4e42c4688200ba3f5dc5cbc799c45c8ad2a05808..4a5ebfd302c3f0fce1204d5f208a439660477ac6 100644 (file)
@@ -257,7 +257,7 @@ public:
   /**
    * Destructor: delete the list
    */
-  ~CDList() throw(AssertionException) {
+  ~CDList() {
     this->destroy();
 
     if(this->d_callDestructor) {
index e2fdcf9bded0c750445187a7b3db0ef1e8660174..21465181c4a815535ad5fb32b50fb1f4aabccac4 100644 (file)
@@ -140,7 +140,7 @@ public:
   /**
    * Destructor - call destroy() method
    */
-  ~CDO() throw(AssertionException) { destroy(); }
+  ~CDO() { destroy(); }
 
   /**
    * Set the data in the CDO.  First call makeCurrent.
index e89c1b528d34af4f7d7a4b30537c798dc1577f73..90816d5421caff8567c398f44b22c93781776ed8 100644 (file)
@@ -452,7 +452,7 @@ public:
   /**
    * Destructor: delete the map
    */
-  ~CDTrailHashMap() throw(AssertionException) {
+  ~CDTrailHashMap() {
     this->destroy();
     delete d_trailMap;
   }
index 9c0416ce8ac5ad421fafbce6f3030f40b0cc637d..8f151ad08dee194bfc2a3fe99c4db9ae834e4e0b 100644 (file)
@@ -35,7 +35,7 @@ Context::Context() : d_pCNOpre(NULL), d_pCNOpost(NULL) {
 }
 
 
-Context::~Context() throw(AssertionException) {
+Context::~Context() {
   // Delete all Scopes
   popto(0);
 
@@ -306,7 +306,7 @@ ContextNotifyObj::ContextNotifyObj(Context* pContext, bool preNotify) {
 }
 
 
-ContextNotifyObj::~ContextNotifyObj() throw(AssertionException) {
+ContextNotifyObj::~ContextNotifyObj() {
   if(d_pCNOnext != NULL) {
     d_pCNOnext->d_ppCNOprev = d_ppCNOprev;
   }
index 4f45e8954cff4b8a548f33fdf364b65c15605d88..f7707bdec927a65c3a7a00c860e0468683386a51 100644 (file)
@@ -152,7 +152,7 @@ public:
   /**
    * Destructor: pop all scopes, delete ContextMemoryManager
    */
-  ~Context() throw(AssertionException);
+  ~Context();
 
   /**
    * Return the current (top) scope
@@ -277,7 +277,7 @@ public:
    * Destructor: Restore all of the objects in ContextObjList.  Defined inline
    * below.
    */
-  ~Scope() throw(AssertionException);
+  ~Scope();
 
   /**
    * Get the Context for this Scope
@@ -615,7 +615,7 @@ public:
   /**
    * Destructor does nothing: subclass must explicitly call destroy() instead.
    */
-  virtual ~ContextObj() throw(AssertionException) {}
+  virtual ~ContextObj() {}
 
   /**
    * If you want to allocate a ContextObj object on the heap, use this
@@ -711,7 +711,7 @@ public:
   /**
    * Destructor: removes object from list
    */
-  virtual ~ContextNotifyObj() throw(AssertionException);
+  virtual ~ContextNotifyObj();
 
 };/* class ContextNotifyObj */
 
@@ -725,7 +725,7 @@ inline void ContextObj::makeSaveRestorePoint() throw(AssertionException) {
   update();
 }
 
-inline Scope::~Scope() throw(AssertionException) {
+inline Scope::~Scope() {
   // Call restore() method on each ContextObj object in the list.
   // Note that it is the responsibility of restore() to return the
   // next item in the list.
index 0a6b08b6a7657e0c94c0a7150599512b9f87737d..2dc2c03bb778fc5ced7fe9dff59bca845117abd6 100644 (file)
@@ -63,7 +63,7 @@ ContextMemoryManager::ContextMemoryManager() : d_indexChunkList(0) {
 }
 
 
-ContextMemoryManager::~ContextMemoryManager() throw() {
+ContextMemoryManager::~ContextMemoryManager() {
   // Delete all chunks
   while(!d_chunkList.empty()) {
     free(d_chunkList.back());
index 99b448cf25a91e26c48b38747bf96122e1872710..673fb90988122f2ba6292d878ded4e2b58d23809 100644 (file)
@@ -118,7 +118,7 @@ public:
   /**
    * Destructor - deletes all memory in all regions
    */
-  ~ContextMemoryManager() throw();
+  ~ContextMemoryManager();
 
   /**
    * Allocate size bytes from the current region
@@ -161,7 +161,7 @@ public:
   ContextMemoryAllocator(ContextMemoryManager* mm) throw() : d_mm(mm) {}
   template <class U>
   ContextMemoryAllocator(const ContextMemoryAllocator<U>& alloc) throw() : d_mm(alloc.getCMM()) {}
-  ~ContextMemoryAllocator() throw() {}
+  ~ContextMemoryAllocator() {}
 
   ContextMemoryManager* getCMM() const { return d_mm; }
   T* address(T& v) const { return &v; }
index b737c40ae6cc081b98858687e3849b418c1e16a1..4b482c9b0b9bfb76af103cb913af7d3b7a4cdd38 100644 (file)
@@ -50,7 +50,7 @@ public:
     d_offset(ctxt, 0) {
   }
 
-  ~StackingVector() throw() { }
+  ~StackingVector() { }
 
   /**
    * Return a value from the vector.  If n is not a key in
index 998294da34b0bf0d4da6f6eb549efba651b37ef1..c9bfb75a421ebcafbc85bc6dd91adef25a6567d7 100644 (file)
@@ -308,7 +308,7 @@ public:
    * Destructor. If ref_count is true it will decrement the reference count
    * and, if zero, collect the NodeValue.
    */
-  ~NodeTemplate() throw(AssertionException);
+  ~NodeTemplate();
 
   /**
    * Return the null node.
@@ -1089,7 +1089,7 @@ NodeTemplate<ref_count>::NodeTemplate(const Expr& e) {
 }
 
 template <bool ref_count>
-NodeTemplate<ref_count>::~NodeTemplate() throw(AssertionException) {
+NodeTemplate<ref_count>::~NodeTemplate() {
   Assert(d_nv != NULL, "Expecting a non-NULL expression value!");
   if(ref_count) {
     // shouldn't ever fail
index 46fdaa143b3f7ebd37229cd2c9c710dd7e7da0a5..0cada0df222d00eb6c31b009e3fa6240aa35e525 100644 (file)
@@ -116,7 +116,7 @@ public:
    * Destructor. If ref_count is true it will decrement the reference count
    * and, if zero, collect the NodeValue.
    */
-  ~TypeNode() throw(AssertionException);
+  ~TypeNode();
 
   /**
    * Assignment operator for nodes, copies the relevant information from node
@@ -785,7 +785,7 @@ inline TypeNode::TypeNode(const TypeNode& typeNode) {
   d_nv->inc();
 }
 
-inline TypeNode::~TypeNode() throw(AssertionException) {
+inline TypeNode::~TypeNode() {
   Assert(d_nv != NULL, "Expecting a non-NULL expression value!");
   d_nv->dec();
 }
index 54e3f2e8ba6a54855c443d9ee9c4760c0d949c1c..368a79b80d99f499085c88c96591545112d59657 100644 (file)
@@ -39,7 +39,7 @@ BVMinisatSatSolver::BVMinisatSatSolver(StatisticsRegistry* registry, context::Co
 }
 
 
-BVMinisatSatSolver::~BVMinisatSatSolver() throw(AssertionException) {
+BVMinisatSatSolver::~BVMinisatSatSolver() {
   delete d_minisat;
   delete d_minisatNotify;
 }
@@ -309,5 +309,3 @@ void toSatClause< BVMinisat::Solver> (const BVMinisat::Solver::TClause& minisat_
 }
 
 }
-
-
index 56a7c61e2c10043e23ea96f191142a1a372409f8..732bd0313f4f2470163d3f10ec2028a752efa76f 100644 (file)
@@ -72,7 +72,7 @@ protected:
 public:
 
   BVMinisatSatSolver(StatisticsRegistry* registry, context::Context* mainSatContext, const std::string& name = "");
-  ~BVMinisatSatSolver() throw(AssertionException);
+  virtual ~BVMinisatSatSolver();
 
   void setNotify(Notify* notify);
 
index d8f25a78640303d4043ee3dbfc0becab1ad53cde..eff079cc9a705d04f1eb5e675dd1a56632cab575 100644 (file)
@@ -45,7 +45,7 @@ CryptoMinisatSolver::CryptoMinisatSolver(StatisticsRegistry* registry,
 }
 
 
-CryptoMinisatSolver::~CryptoMinisatSolver() throw(AssertionException) {
+CryptoMinisatSolver::~CryptoMinisatSolver() {
   delete d_solver;
 }
 
index 54d52af0e7a18128fc00fdda900aa72e3204124c..7ad861defb0cc09b350e8d235872628e3d1003ce 100644 (file)
@@ -38,7 +38,7 @@ private:
 public:
   CryptoMinisatSolver(StatisticsRegistry* registry,
                       const std::string& name = "");
-  ~CryptoMinisatSolver() throw(AssertionException);
+  virtual ~CryptoMinisatSolver();
 
   ClauseId addClause(SatClause& clause, bool removable);
   ClauseId addXorClause(SatClause& clause, bool rhs, bool removable);
@@ -132,7 +132,3 @@ public:
 } // CVC4
 
 #endif // CVC4_USE_CRYPTOMINISAT
-
-
-
-
index ff726e29910d1567f3d662a1ddafeed134f5d588..a88a872c83cb9da555c68b073cf0f89f9b57b454 100644 (file)
@@ -38,7 +38,7 @@ MinisatSatSolver::MinisatSatSolver(StatisticsRegistry* registry) :
   d_statistics(registry)
 {}
 
-MinisatSatSolver::~MinisatSatSolver() throw()
+MinisatSatSolver::~MinisatSatSolver()
 {
   delete d_minisat;
 }
index ec5297bb7ee0a8b0887b057bcd71081f4158a430..1627a657560fbcdd2ec99ef84717c77e3980edf0 100644 (file)
@@ -29,7 +29,7 @@ class MinisatSatSolver : public DPLLSatSolverInterface {
 public:
 
   MinisatSatSolver(StatisticsRegistry* registry);
-  ~MinisatSatSolver() throw();
+  virtual ~MinisatSatSolver();
 ;
 
   static SatVariable     toSatVariable(Minisat::Var var);
index 81fb9424221e497c0154b1a61a9f32324a75c44a..4afaea5a08195a4b8eb3b889f564e271a1f4c61b 100644 (file)
@@ -43,7 +43,7 @@ class SatSolver {
 public:
 
   /** Virtual destructor */
-  virtual ~SatSolver() throw(AssertionException) { }
+  virtual ~SatSolver() { }
 
   /** Assert a clause in the solver. */
   virtual ClauseId addClause(SatClause& clause,
@@ -99,7 +99,7 @@ public:
 class BVSatSolverInterface: public SatSolver {
 public:
 
-  virtual ~BVSatSolverInterface() throw(AssertionException) {}
+  virtual ~BVSatSolverInterface() {}
   /** Interface for notifications */
   class Notify {
   public:
index 45217258649c45e903d80c8deb0c1c29e872962d..923e1d8c5f902809b5eff85ce653bfafb578e1dd 100644 (file)
@@ -242,7 +242,7 @@ public:
    */
   bool addDisequality(TNode a, TNode b, TNode reason); 
   void getConflict(std::vector<TNode>& conflict);
-  virtual ~InequalityGraph() throw(AssertionException) {}
+  virtual ~InequalityGraph() {}
   /** 
    * Check that the currently asserted disequalities that have not been split on
    * are still true in the current model. 
index 0dc6cc7a192848c16c1ec9a4b895355912f4cc4f..b78109dfbd3dcbed0a6b9c9850684c1b1d544db5 100644 (file)
@@ -37,7 +37,7 @@ SharedTermsDatabase::SharedTermsDatabase(TheoryEngine* theoryEngine, context::Co
   smtStatisticsRegistry()->registerStat(&d_statSharedTerms);
 }
 
-SharedTermsDatabase::~SharedTermsDatabase() throw(AssertionException)
+SharedTermsDatabase::~SharedTermsDatabase()
 {
   smtStatisticsRegistry()->unregisterStat(&d_statSharedTerms);
 }
index c108122ef3e53ccf2e9643fe61cdac54fc3d1c9f..cc8959165743678d03051406603650579c8b0e79 100644 (file)
@@ -149,7 +149,7 @@ private:
 public:
 
   SharedTermsDatabase(TheoryEngine* theoryEngine, context::Context* context);
-  ~SharedTermsDatabase() throw(AssertionException);
+  ~SharedTermsDatabase();
 
   /**
    * Asserts the equality to the shared terms database,
index 09d348584ca3476665c855311ff7976953b0b251..f98ad556f2ed3baa85b42fa15f36c78b6bc1c845 100644 (file)
@@ -97,7 +97,7 @@ void EqualityEngine::init() {
   d_freshMergeReasonType = eq::NUMBER_OF_MERGE_REASONS;
 }
 
-EqualityEngine::~EqualityEngine() throw(AssertionException) {
+EqualityEngine::~EqualityEngine() {
   free(d_triggerDatabase);
 }
 
index 843e7ce7fce12928dd263045b000e42aa0b60c3e..18e83bd1ada8a25986d2e1e4fe6d396f3304d59c 100644 (file)
@@ -186,7 +186,7 @@ public:
   /**
    * Just a destructor.
    */
-  virtual ~EqualityEngine() throw(AssertionException);
+  virtual ~EqualityEngine();
 
   /**
    * Set the master equality engine for this one. Master engine will get copies of all