accel-sim / accel-sim-framework

This is the top-level repository for the Accel-Sim framework.
https://accel-sim.github.io
Other
306 stars 117 forks source link

Implement move semantics and disable copy semantics for the new PipeReader #342

Closed FJShen closed 1 month ago

FJShen commented 1 month ago

The PipeReader introduced in PR #340 holds a file handle to a unique Linux anonymous pipe. Therefore, we should prevent someone from trying to 'copy' a PipeReader class object. While there is currently no such use case in the repository, it is important to implement preventative measures before someone shoots himself in the foot.