On 10/26/21 18:32, Tao Zhou wrote: >> + if (params->stop_us) >> + osnoise_set_stop_us(tool->context, params->stop_us); > osnoise_set_stop_us() also need to check return value feels by me. > >> + >> + if (params->stop_total_us) >> + osnoise_set_stop_total_us(tool->context, params->stop_total_us); > The same here. Also need to check the return value. > > PATCH 10 also has these two places(call the two functions) that not check > the return value. > > Or I am wrong. You are right, reviewing that. Thanks! -- Daniel > > > Thanks, > Tao >
