04.03.2015 15:16, 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. >> >> Signed-off-by: Michael Tokarev <m...@tls.msk.ru> > > Juan/Amit: There's a space at the end of a line that got in, so > that just needs fixing in the merge; it's after the 'break;'
I already fixed it in my tree two days ago. > but the patch is a big improvement on what's there: > > Reviewed-by: Dr. David Alan Gilbert <dgilb...@redhat.com> > > also tested on an RDMA system. Thank you! I think it's okay to merge it using the -trivial tree, I'm about to send a pull request today. /mjt
