Skip to content

Returning 0 in ipmiAsynPortDriver::writeInt32 and writeFloat64

The last line of these two functions is

    return (asynStatus)0 ;

In the beginning of the functions the status variable is set to the return value of setXXXParam() and then status is returned if there was additional processing (other parameters set or monitor values changed).

Shouldn't we also return status when nothing extra was done? Or at least return asynSuccess instead of 0