Removed JEvent use check when it is returning to pool

CMakeLists.txt

@@ -1,7 +1,7 @@
 cmake_minimum_required(VERSION 3.16)
 cmake_policy(SET CMP0074 NEW) # find_package() uses <PackageName>_ROOT implicit hints
 
-project(jana2 VERSION 2.3.2)
+project(jana2 VERSION 2.3.3)
 
 set(CMAKE_POSITION_INDEPENDENT_CODE ON)   # Enable -fPIC for all targets
 

docs/Download.md

@@ -10,6 +10,25 @@
 - [See online doxygen documentation](http://www.jlab.org/JANA/jana_doc_latest/index.html)
 - [Download doxygen documentation](http://www.jlab.org/JANA/jana_doc_latest.tar.gz)
 
+### 2.3.3
+
+#### Bugfixes
+* Fix problem with user-defined factory generators (#366)
+* JEventProcessor::Process() called before BeginRun() (#367)
+* Lock overwrite in RootFillLock() (#369)
+* JFactory::Finish() is called (#377)
+
+#### Features
+* JTopologyBuilder supports topologies with arbitrarily nested levels (#346)
+* Barrier events are back (#371)
+
+#### Refactoring
+* Improved log output (#368)
+* JTest uses new-style component interfaces (#374)
+* JArrows now fire on individual events (#375, #378)
+
+- [See release on GitHub](https://github.com/JeffersonLab/JANA2/releases/tag/v2.3.3)
+
 ### 2.3.2
 This release includes the following:
 

src/libraries/JANA/Topology/JEventPool.h

@@ -98,11 +98,6 @@ class JEventPool {
             // Do any necessary teardown within the item itself
             item->Clear();
         }
-        auto use_count = item->shared_from_this().use_count();
-        if (use_count > 2) {
-            // Use count should be 2 because there's the shared_ptr in `m_owned_events`, and there's the temporary shared_ptr created just now
-            throw JException("Attempted to return a JEvent to the pool while it is still being used! use_count=%d", use_count);
-        }
 
         LocalPool& pool = *(m_pools[location]);