Skip to content

Commit

Permalink
Remove obsolete event_pool data member from JTopologyBuilder
Browse files Browse the repository at this point in the history
  • Loading branch information
nathanwbrei committed Dec 5, 2024
1 parent c216891 commit 1a5c914
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 6 deletions.
5 changes: 0 additions & 5 deletions src/libraries/JANA/Topology/JTopologyBuilder.cc
Original file line number Diff line number Diff line change
Expand Up @@ -28,9 +28,6 @@ JTopologyBuilder::~JTopologyBuilder() {
for (auto pool : pools) {
delete pool;
}
if (event_pool != nullptr) {
delete event_pool;
}
}

std::string JTopologyBuilder::print_topology() {
Expand Down Expand Up @@ -92,8 +89,6 @@ void JTopologyBuilder::create_topology() {
mapping.initialize(static_cast<JProcessorMapping::AffinityStrategy>(m_affinity),
static_cast<JProcessorMapping::LocalityStrategy>(m_locality));

event_pool = new JEventPool(m_components, m_max_inflight_events, m_location_count);

if (m_configure_topology) {
m_configure_topology(*this);
LOG_WARN(GetLogger()) << "Found custom topology configurator! Modified arrow topology is: \n" << print_topology() << LOG_END;
Expand Down
1 change: 0 additions & 1 deletion src/libraries/JANA/Topology/JTopologyBuilder.h
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,6 @@ class JTopologyBuilder : public JService {

// Things that probably shouldn't be here
std::function<void(JTopologyBuilder&)> m_configure_topology;
JEventPool* event_pool = nullptr; // TODO: Move into pools eventually
JProcessorMapping mapping;

public:
Expand Down

0 comments on commit 1a5c914

Please sign in to comment.