19.02.2015 13:25, Dr. David Alan Gilbert wrote:
> * Michael Tokarev (m...@tls.msk.ru) wrote:
>> Do not check for rdma->host being empty twice.  This removes a large
>> "if" block, so code indentation is changed.  While at it, remove an
>> ugly goto from the loop, replacing it with a cleaner if logic.  And
>> finally, there's no need to initialize `ret' variable since is always
>> has a value.
> 
> This looks OK; have you got RDMA hardware to test with, if not I can
> give it a go.

No, I don't have any RDMA hw, I only compile-tested this code, and
double-checked the changes does not change behavour.

Thanks,

/mjt

Reply via email to