Thanks for doing this. But that is not what I meant. Struct Data should only have basic data types which require to be sending between threads. What is implemented here is an immediate struct definition in a class which will be passed and is literally the same as a class and has the only overhead of a struct defined in the class. Please use Data as the data and remove the functions from the struct Data. I think the concept of having the struct Data is not clarified, let me know if I need to explain mor.
Can we clear this alarm when it is raised? If no, do we need this else statement? Because if the alarm has not been raised, we do not need to clear it either.
Should AND this bit off from register so we don't interfere with other bits that may have been set by other drivers (e.g. bubble detector). Should be something like fpgaActuatorSetPoints.fpgaSensorTest &= ~FPGA_BLOOD_LEAK_ZERO_CMD;
Fixed. If Crucible doesn't show you the latest update please drag the comment slider (which is on top of the current file view) right knob to the right end under the last commit.
Don't think prepend("/") is needed here since according to https://doc.qt.io/qt-5/qfileinfo.html#absolutePath, "On Unix the absolute path will always begin with the root, '/', directory."
