remove log infos

niknal357 · niknal357 · commit 4fb7c683a4e6 · 2025-10-25T04:02:37.000+02:00
diff --git a/src/backend/evaluator/busInterfacePassthrough.h b/src/backend/evaluator/busInterfacePassthrough.h
@@ -16,7 +16,6 @@ class BusInterfacePassthrough {
     blockDataManager(blockDataManager) {}
 
     void addGate(SimPauseGuard& pauseGuard, const BlockType blockType, const middle_id_t gateId) {
-		logInfo("BusInterfacePassthrough::addGate {} {}", "", blockType, gateId);
         BlockData* blockData = blockDataManager.getBlockData(blockType);
         if (blockData->isBus()) {
             busInterfaces[gateId] = blockType;
@@ -26,7 +25,6 @@ class BusInterfacePassthrough {
     }
 
     void removeGate(SimPauseGuard& pauseGuard, const middle_id_t gateId) {
-		logInfo("BusInterfacePassthrough::removeGate {}", "", gateId);
         if (omittedConnections.contains(gateId)) {
             std::vector<EvalConnection> omittedConns = omittedConnections[gateId];
             for (const EvalConnection& conn : omittedConns) {
@@ -58,12 +56,10 @@ class BusInterfacePassthrough {
     }
 
     inline SimPauseGuard beginEdit() {
-		logInfo("BusInterfacePassthrough::beginEdit");
         return simulatorOptimizer.beginEdit();
     }
 
     void endEdit(SimPauseGuard& pauseGuard) {
-		logInfo("BusInterfacePassthrough::endEdit");
         simulatorOptimizer.endEdit(pauseGuard);
     }
 
@@ -122,7 +118,6 @@ class BusInterfacePassthrough {
     }
 
     void makeConnection(SimPauseGuard& pauseGuard, EvalConnection connection) {
-		logInfo("BusInterfacePassthrough::makeConnection {}", "", connection.toString());
         if (busInterfaces.contains(connection.source.gateId) ||
             busInterfaces.contains(connection.destination.gateId)) {
             omittedConnections[connection.source.gateId].push_back(connection);
@@ -134,7 +129,6 @@ class BusInterfacePassthrough {
     }
 
     void removeConnection(SimPauseGuard& pauseGuard, EvalConnection connection) {
-		logInfo("BusInterfacePassthrough::removeConnection {}", "", connection.toString());
         if (busInterfaces.contains(connection.source.gateId) ||
             busInterfaces.contains(connection.destination.gateId)) {
             dirtySimulatorIds.push_back(0);
diff --git a/src/backend/evaluator/evalSimulator.h b/src/backend/evaluator/evalSimulator.h
@@ -14,19 +14,15 @@ class EvalSimulator {
 	) :
 		gateSubstituter(evalConfig, middleIdProvider, dirtySimulatorIds, dirtyMiddleIds, blockDataManager) {}
 	inline SimPauseGuard beginEdit() {
-		logInfo("EvalSimulator::beginEdit");
 		return gateSubstituter.beginEdit();
 	}
 	inline void endEdit(SimPauseGuard& pauseGuard) {
-		logInfo("EvalSimulator::endEdit");
 		gateSubstituter.endEdit(pauseGuard);
 	}
 	inline void addGate(SimPauseGuard& pauseGuard, const BlockType blockType, const middle_id_t gateId) {
-		logInfo("EvalSimulator::addGate {} {}", "", blockType, gateId);
 		gateSubstituter.addGate(pauseGuard, blockType, gateId);
 	}
 	inline void removeGate(SimPauseGuard& pauseGuard, const middle_id_t gateId) {
-		logInfo("EvalSimulator::removeGate {}", "", gateId);
 		gateSubstituter.removeGate(pauseGuard, gateId);
 	}
 	inline logic_state_t getState(EvalConnectionPoint point) const {
@@ -51,14 +47,9 @@ class EvalSimulator {
 		gateSubstituter.setState(point, state);
 	}
 	inline void makeConnection(SimPauseGuard& pauseGuard, EvalConnection connection) {
-		logInfo("EvalSimulator::makeConnection {}", "", connection.toString());
-		if (connection.toString() == "EC( (32, 0) -> (24, 0) )") {
-			logInfo("BREAK");
-		}
 		gateSubstituter.makeConnection(pauseGuard, connection);
 	}
 	inline void removeConnection(SimPauseGuard& pauseGuard, EvalConnection connection) {
-		logInfo("EvalSimulator::removeConnection {}", "", connection.toString());
 		gateSubstituter.removeConnection(pauseGuard, connection);
 	}
 	inline double getAverageTickrate() const {
diff --git a/src/backend/evaluator/evaluator.cpp b/src/backend/evaluator/evaluator.cpp
@@ -95,44 +95,37 @@ void Evaluator::makeEditInPlace(SimPauseGuard& pauseGuard, eval_circuit_id_t eva
 	}
 	const BlockContainer* blockContainer = circuit->getBlockContainer();
 
-	logInfo("EVAL START DIFF: {}", "", evalCircuitId);
 	const std::vector<Difference::Modification>& modifications = difference->getModifications();
 	for (const Difference::Modification& modification : modifications) {
 		const auto& [modificationType, modificationData] = modification;
 		switch (modificationType) {
 		case Difference::ModificationType::REMOVED_BLOCK: {
 			const auto& [position, orientation, blockType] = std::get<Difference::block_modification_t>(modificationData);
-			logInfo("REMOVED_BLOCK {}, {}, {}", "", position, orientation, blockType);
 			edit_removeBlock(pauseGuard, evalCircuitId, diffCache, position, orientation, blockType);
 			break;
 		}
 		case Difference::ModificationType::PLACE_BLOCK: {
 			const auto& [position, orientation, blockType] = std::get<Difference::block_modification_t>(modificationData);
-			logInfo("PLACE_BLOCK {}, {}, {}", "", position, orientation, blockType);
 			edit_placeBlock(pauseGuard, evalCircuitId, diffCache, position, orientation, blockType);
 			break;
 		}
 		case Difference::ModificationType::MOVE_BLOCK: {
 			const auto& [curPosition, curOrientation, newPosition, newOrientation, finalMove] = std::get<Difference::move_modification_t>(modificationData);
-			logInfo("MOVE_BLOCK {}, {}, {}, {}", "", curPosition, curOrientation, newPosition, newOrientation);
 			edit_moveBlock(pauseGuard, evalCircuitId, diffCache, curPosition, curOrientation, newPosition, newOrientation, finalMove);
 			break;
 		}
 		case Difference::ModificationType::REMOVED_CONNECTION: {
 			const auto& [outputBlockPosition, outputPosition, inputBlockPosition, inputPosition] = std::get<Difference::connection_modification_t>(modificationData);
-			logInfo("REMOVED_CONNECTION {}, {}, {}, {}", "", outputBlockPosition, outputPosition, inputBlockPosition, inputPosition);
 			edit_removeConnection(pauseGuard, evalCircuitId, diffCache, blockContainer, outputBlockPosition, outputPosition, inputBlockPosition, inputPosition);
 			break;
 		}
 		case Difference::ModificationType::CREATED_CONNECTION: {
 			const auto& [outputBlockPosition, outputPosition, inputBlockPosition, inputPosition] = std::get<Difference::connection_modification_t>(modificationData);
-			logInfo("CREATED_CONNECTION {}, {}, {}, {}", "", outputBlockPosition, outputPosition, inputBlockPosition, inputPosition);
 			edit_createConnection(pauseGuard, evalCircuitId, diffCache, blockContainer, outputBlockPosition, outputPosition, inputBlockPosition, inputPosition);
 			break;
 		}
 		}
 	}
-	logInfo("EVAL END DIFF: {}", "", evalCircuitId);
 }
 
 void Evaluator::edit_removeBlock(
@@ -205,7 +198,6 @@ void Evaluator::edit_placeBlock(
 	}
 
 	middle_id_t gateId = middleIdProvider.getNewId();
-	logInfo("gateId: {}", "", gateId);
 	evalSimulator->addGate(pauseGuard, blockType, gateId);
 	middleIdToEvalPositionMap[gateId] = { position, evalCircuitId };
 	EvalCircuit* evalCircuit = evalCircuitContainer.getCircuit(evalCircuitId);
@@ -292,7 +284,6 @@ void Evaluator::edit_createConnection(
 		return;
 	}
 	EvalConnection connection(outputPoint.value(), inputPoint.value());
-	logInfo("gateId > gateId: {}", "", connection.toString());
 	if (!circuitPortDependencies.empty() || !circuitNodeDependencies.empty()) {
 		interCircuitConnections.push_back({ connection, circuitPortDependencies, circuitNodeDependencies });
 	}
diff --git a/src/backend/evaluator/logicSimulator.cpp b/src/backend/evaluator/logicSimulator.cpp
@@ -423,13 +423,11 @@ simulator_id_t LogicSimulator::addGate(const BlockType blockType) {
 		logError("Cannot add gate of type {}", "LogicSimulator::addGate", (unsigned int)blockType);
 		return 0;
 	}
-	logInfo("LogicSimulator::addGate ({}) -> simID: {}", "", blockType, simulatorId);
 
 	return simulatorId;
 }
 
 void LogicSimulator::removeGate(simulator_id_t simulatorId) {
-	logInfo("LogicSimulator::removeGate ({})", "", simulatorId);
 	auto locationIt = gateLocations.find(simulatorId);
 	if (locationIt == gateLocations.end()) {
 		logError("Cannot remove gate: not found " + std::to_string(simulatorId), "LogicSimulator::removeGate");
@@ -501,7 +499,6 @@ void LogicSimulator::removeGate(simulator_id_t simulatorId) {
 }
 
 void LogicSimulator::makeConnection(simulator_id_t sourceId, connection_port_id_t sourcePort, simulator_id_t destinationId, connection_port_id_t destinationPort) {
-	logInfo("LogicSimulator::makeConnection {} {} {} {}", "", sourceId, sourcePort, destinationId, destinationPort);
 	std::optional<simulator_id_t> actualSourceId = getOutputPortId(sourceId, sourcePort);
 
 	if (!actualSourceId.has_value()) {
@@ -513,7 +510,6 @@ void LogicSimulator::makeConnection(simulator_id_t sourceId, connection_port_id_
 }
 
 void LogicSimulator::removeConnection(simulator_id_t sourceId, connection_port_id_t sourcePort, simulator_id_t destinationId, connection_port_id_t destinationPort) {
-	logInfo("LogicSimulator::makeConnection {} {} {} {}", "", sourceId, sourcePort, destinationId, destinationPort);
 	std::optional<simulator_id_t> actualSourceId = getOutputPortId(sourceId, sourcePort);
 	if (!actualSourceId.has_value()) {
 		logError("Cannot resolve actual source ID for disconnection", "LogicSimulator::removeConnection");
@@ -523,8 +519,6 @@ void LogicSimulator::removeConnection(simulator_id_t sourceId, connection_port_i
 }
 
 void LogicSimulator::endEdit() {
-	logInfo("LogicSimulator::endEdit");
-
 	for (auto& gate : junctions) gate.doubleTick(statesA, statesB);
 	regenerateJobs();
 }

Original file line number	Diff line number	Diff line change
`@@ -16,7 +16,6 @@ class BusInterfacePassthrough {`
`16`	`16`	`blockDataManager(blockDataManager) {}`
`17`	`17`
`18`	`18`	`void addGate(SimPauseGuard& pauseGuard, const BlockType blockType, const middle_id_t gateId) {`
`19`		`- logInfo("BusInterfacePassthrough::addGate {} {}", "", blockType, gateId);`
`20`	`19`	`BlockData* blockData = blockDataManager.getBlockData(blockType);`
`21`	`20`	`if (blockData->isBus()) {`
`22`	`21`	`busInterfaces[gateId] = blockType;`
`@@ -26,7 +25,6 @@ class BusInterfacePassthrough {`
`26`	`25`	`}`
`27`	`26`
`28`	`27`	`void removeGate(SimPauseGuard& pauseGuard, const middle_id_t gateId) {`
`29`		`- logInfo("BusInterfacePassthrough::removeGate {}", "", gateId);`
`30`	`28`	`if (omittedConnections.contains(gateId)) {`
`31`	`29`	`std::vector<EvalConnection> omittedConns = omittedConnections[gateId];`
`32`	`30`	`for (const EvalConnection& conn : omittedConns) {`
`@@ -58,12 +56,10 @@ class BusInterfacePassthrough {`
`58`	`56`	`}`
`59`	`57`
`60`	`58`	`inline SimPauseGuard beginEdit() {`
`61`		`- logInfo("BusInterfacePassthrough::beginEdit");`
`62`	`59`	`return simulatorOptimizer.beginEdit();`
`63`	`60`	`}`
`64`	`61`
`65`	`62`	`void endEdit(SimPauseGuard& pauseGuard) {`
`66`		`- logInfo("BusInterfacePassthrough::endEdit");`
`67`	`63`	`simulatorOptimizer.endEdit(pauseGuard);`
`68`	`64`	`}`
`69`	`65`
`@@ -122,7 +118,6 @@ class BusInterfacePassthrough {`
`122`	`118`	`}`
`123`	`119`
`124`	`120`	`void makeConnection(SimPauseGuard& pauseGuard, EvalConnection connection) {`
`125`		`- logInfo("BusInterfacePassthrough::makeConnection {}", "", connection.toString());`
`126`	`121`	`if (busInterfaces.contains(connection.source.gateId) \|\|`
`127`	`122`	`busInterfaces.contains(connection.destination.gateId)) {`
`128`	`123`	`omittedConnections[connection.source.gateId].push_back(connection);`
`@@ -134,7 +129,6 @@ class BusInterfacePassthrough {`
`134`	`129`	`}`
`135`	`130`
`136`	`131`	`void removeConnection(SimPauseGuard& pauseGuard, EvalConnection connection) {`
`137`		`- logInfo("BusInterfacePassthrough::removeConnection {}", "", connection.toString());`
`138`	`132`	`if (busInterfaces.contains(connection.source.gateId) \|\|`
`139`	`133`	`busInterfaces.contains(connection.destination.gateId)) {`
`140`	`134`	`dirtySimulatorIds.push_back(0);`
Original file line number	Diff line number	Diff line change
`@@ -14,19 +14,15 @@ class EvalSimulator {`
`14`	`14`	`) :`
`15`	`15`	`gateSubstituter(evalConfig, middleIdProvider, dirtySimulatorIds, dirtyMiddleIds, blockDataManager) {}`
`16`	`16`	`inline SimPauseGuard beginEdit() {`
`17`		`- logInfo("EvalSimulator::beginEdit");`
`18`	`17`	`return gateSubstituter.beginEdit();`
`19`	`18`	`}`
`20`	`19`	`inline void endEdit(SimPauseGuard& pauseGuard) {`
`21`		`- logInfo("EvalSimulator::endEdit");`
`22`	`20`	`gateSubstituter.endEdit(pauseGuard);`
`23`	`21`	`}`
`24`	`22`	`inline void addGate(SimPauseGuard& pauseGuard, const BlockType blockType, const middle_id_t gateId) {`
`25`		`- logInfo("EvalSimulator::addGate {} {}", "", blockType, gateId);`
`26`	`23`	`gateSubstituter.addGate(pauseGuard, blockType, gateId);`
`27`	`24`	`}`
`28`	`25`	`inline void removeGate(SimPauseGuard& pauseGuard, const middle_id_t gateId) {`
`29`		`- logInfo("EvalSimulator::removeGate {}", "", gateId);`
`30`	`26`	`gateSubstituter.removeGate(pauseGuard, gateId);`
`31`	`27`	`}`
`32`	`28`	`inline logic_state_t getState(EvalConnectionPoint point) const {`
`@@ -51,14 +47,9 @@ class EvalSimulator {`
`51`	`47`	`gateSubstituter.setState(point, state);`
`52`	`48`	`}`
`53`	`49`	`inline void makeConnection(SimPauseGuard& pauseGuard, EvalConnection connection) {`
`54`		`- logInfo("EvalSimulator::makeConnection {}", "", connection.toString());`
`55`		`- if (connection.toString() == "EC( (32, 0) -> (24, 0) )") {`
`56`		`- logInfo("BREAK");`
`57`		`- }`
`58`	`50`	`gateSubstituter.makeConnection(pauseGuard, connection);`
`59`	`51`	`}`
`60`	`52`	`inline void removeConnection(SimPauseGuard& pauseGuard, EvalConnection connection) {`
`61`		`- logInfo("EvalSimulator::removeConnection {}", "", connection.toString());`
`62`	`53`	`gateSubstituter.removeConnection(pauseGuard, connection);`
`63`	`54`	`}`
`64`	`55`	`inline double getAverageTickrate() const {`
Original file line number	Diff line number	Diff line change
`@@ -95,44 +95,37 @@ void Evaluator::makeEditInPlace(SimPauseGuard& pauseGuard, eval_circuit_id_t eva`
`95`	`95`	`}`
`96`	`96`	`const BlockContainer* blockContainer = circuit->getBlockContainer();`
`97`	`97`
`98`		`- logInfo("EVAL START DIFF: {}", "", evalCircuitId);`
`99`	`98`	`const std::vector<Difference::Modification>& modifications = difference->getModifications();`
`100`	`99`	`for (const Difference::Modification& modification : modifications) {`
`101`	`100`	`const auto& [modificationType, modificationData] = modification;`
`102`	`101`	`switch (modificationType) {`
`103`	`102`	`case Difference::ModificationType::REMOVED_BLOCK: {`
`104`	`103`	`const auto& [position, orientation, blockType] = std::get<Difference::block_modification_t>(modificationData);`
`105`		`- logInfo("REMOVED_BLOCK {}, {}, {}", "", position, orientation, blockType);`
`106`	`104`	`edit_removeBlock(pauseGuard, evalCircuitId, diffCache, position, orientation, blockType);`
`107`	`105`	`break;`
`108`	`106`	`}`
`109`	`107`	`case Difference::ModificationType::PLACE_BLOCK: {`
`110`	`108`	`const auto& [position, orientation, blockType] = std::get<Difference::block_modification_t>(modificationData);`
`111`		`- logInfo("PLACE_BLOCK {}, {}, {}", "", position, orientation, blockType);`
`112`	`109`	`edit_placeBlock(pauseGuard, evalCircuitId, diffCache, position, orientation, blockType);`
`113`	`110`	`break;`
`114`	`111`	`}`
`115`	`112`	`case Difference::ModificationType::MOVE_BLOCK: {`
`116`	`113`	`const auto& [curPosition, curOrientation, newPosition, newOrientation, finalMove] = std::get<Difference::move_modification_t>(modificationData);`
`117`		`- logInfo("MOVE_BLOCK {}, {}, {}, {}", "", curPosition, curOrientation, newPosition, newOrientation);`
`118`	`114`	`edit_moveBlock(pauseGuard, evalCircuitId, diffCache, curPosition, curOrientation, newPosition, newOrientation, finalMove);`
`119`	`115`	`break;`
`120`	`116`	`}`
`121`	`117`	`case Difference::ModificationType::REMOVED_CONNECTION: {`
`122`	`118`	`const auto& [outputBlockPosition, outputPosition, inputBlockPosition, inputPosition] = std::get<Difference::connection_modification_t>(modificationData);`
`123`		`- logInfo("REMOVED_CONNECTION {}, {}, {}, {}", "", outputBlockPosition, outputPosition, inputBlockPosition, inputPosition);`
`124`	`119`	`edit_removeConnection(pauseGuard, evalCircuitId, diffCache, blockContainer, outputBlockPosition, outputPosition, inputBlockPosition, inputPosition);`
`125`	`120`	`break;`
`126`	`121`	`}`
`127`	`122`	`case Difference::ModificationType::CREATED_CONNECTION: {`
`128`	`123`	`const auto& [outputBlockPosition, outputPosition, inputBlockPosition, inputPosition] = std::get<Difference::connection_modification_t>(modificationData);`
`129`		`- logInfo("CREATED_CONNECTION {}, {}, {}, {}", "", outputBlockPosition, outputPosition, inputBlockPosition, inputPosition);`
`130`	`124`	`edit_createConnection(pauseGuard, evalCircuitId, diffCache, blockContainer, outputBlockPosition, outputPosition, inputBlockPosition, inputPosition);`
`131`	`125`	`break;`
`132`	`126`	`}`
`133`	`127`	`}`
`134`	`128`	`}`
`135`		`- logInfo("EVAL END DIFF: {}", "", evalCircuitId);`
`136`	`129`	`}`
`137`	`130`
`138`	`131`	`void Evaluator::edit_removeBlock(`
`@@ -205,7 +198,6 @@ void Evaluator::edit_placeBlock(`
`205`	`198`	`}`
`206`	`199`
`207`	`200`	`middle_id_t gateId = middleIdProvider.getNewId();`
`208`		`- logInfo("gateId: {}", "", gateId);`
`209`	`201`	`evalSimulator->addGate(pauseGuard, blockType, gateId);`
`210`	`202`	`middleIdToEvalPositionMap[gateId] = { position, evalCircuitId };`
`211`	`203`	`EvalCircuit* evalCircuit = evalCircuitContainer.getCircuit(evalCircuitId);`
`@@ -292,7 +284,6 @@ void Evaluator::edit_createConnection(`
`292`	`284`	`return;`
`293`	`285`	`}`
`294`	`286`	`EvalConnection connection(outputPoint.value(), inputPoint.value());`
`295`		`- logInfo("gateId > gateId: {}", "", connection.toString());`
`296`	`287`	`if (!circuitPortDependencies.empty() \|\| !circuitNodeDependencies.empty()) {`
`297`	`288`	`interCircuitConnections.push_back({ connection, circuitPortDependencies, circuitNodeDependencies });`
`298`	`289`	`}`
Original file line number	Diff line number	Diff line change
`@@ -423,13 +423,11 @@ simulator_id_t LogicSimulator::addGate(const BlockType blockType) {`
`423`	`423`	`logError("Cannot add gate of type {}", "LogicSimulator::addGate", (unsigned int)blockType);`
`424`	`424`	`return 0;`
`425`	`425`	`}`
`426`		`- logInfo("LogicSimulator::addGate ({}) -> simID: {}", "", blockType, simulatorId);`
`427`	`426`
`428`	`427`	`return simulatorId;`
`429`	`428`	`}`
`430`	`429`
`431`	`430`	`void LogicSimulator::removeGate(simulator_id_t simulatorId) {`
`432`		`- logInfo("LogicSimulator::removeGate ({})", "", simulatorId);`
`433`	`431`	`auto locationIt = gateLocations.find(simulatorId);`
`434`	`432`	`if (locationIt == gateLocations.end()) {`
`435`	`433`	`logError("Cannot remove gate: not found " + std::to_string(simulatorId), "LogicSimulator::removeGate");`
`@@ -501,7 +499,6 @@ void LogicSimulator::removeGate(simulator_id_t simulatorId) {`
`501`	`499`	`}`
`502`	`500`
`503`	`501`	`void LogicSimulator::makeConnection(simulator_id_t sourceId, connection_port_id_t sourcePort, simulator_id_t destinationId, connection_port_id_t destinationPort) {`
`504`		`- logInfo("LogicSimulator::makeConnection {} {} {} {}", "", sourceId, sourcePort, destinationId, destinationPort);`
`505`	`502`	`std::optional<simulator_id_t> actualSourceId = getOutputPortId(sourceId, sourcePort);`
`506`	`503`
`507`	`504`	`if (!actualSourceId.has_value()) {`
`@@ -513,7 +510,6 @@ void LogicSimulator::makeConnection(simulator_id_t sourceId, connection_port_id_`
`513`	`510`	`}`
`514`	`511`
`515`	`512`	`void LogicSimulator::removeConnection(simulator_id_t sourceId, connection_port_id_t sourcePort, simulator_id_t destinationId, connection_port_id_t destinationPort) {`
`516`		`- logInfo("LogicSimulator::makeConnection {} {} {} {}", "", sourceId, sourcePort, destinationId, destinationPort);`
`517`	`513`	`std::optional<simulator_id_t> actualSourceId = getOutputPortId(sourceId, sourcePort);`
`518`	`514`	`if (!actualSourceId.has_value()) {`
`519`	`515`	`logError("Cannot resolve actual source ID for disconnection", "LogicSimulator::removeConnection");`
`@@ -523,8 +519,6 @@ void LogicSimulator::removeConnection(simulator_id_t sourceId, connection_port_i`
`523`	`519`	`}`
`524`	`520`
`525`	`521`	`void LogicSimulator::endEdit() {`
`526`		`- logInfo("LogicSimulator::endEdit");`
`527`		`-`
`528`	`522`	`for (auto& gate : junctions) gate.doubleTick(statesA, statesB);`
`529`	`523`	`regenerateJobs();`
`530`	`524`	`}`