The shorter constructor is setting the local* values, I am saving the device name and address. Also, the error member variable is defaulted to 0 in the .h file.
Instead of generic function, recommend adding a specific function something similar to "getACSwitchStatus' or 'getLeaskSensorStatus' and return the values ( ex: leak sensor Enum states?). Do we need to add the application code as well now, say if leak is found, trigger an alarm?
Same comment as other message handling function. Use correct payload type. See signalPauseResumeUF() function for example of how to respond to a UI request.
Yes. The other alarms are handled in different drivers or modes. The above alarms are here to make sure HD sends the fill command after the bottles have been replaced.