[Fwd: Re: [PATCH] Minor cleanup of libnet_LookupName_recv] || 2nd reviewer, please
Ralph Böhme
slow at samba.org
Mon Feb 26 13:33:54 UTC 2018
Hi Swen,
On Mon, Feb 26, 2018 at 02:10:28PM +0100, Swen Schillig via samba-technical wrote:
> A second review would be highly appreciated on the below.
>
> I added the patch file for convenience.
>
> Thanks in advance.
thanks for the cleanup, but I have one nitpick.
Why do you use unlikely() in an error check of a code path that is probably not
performance critical?
If others don't convince me that this is good practice: nack, sorry.
-slow
--
Ralph Boehme, Samba Team https://samba.org/
Samba Developer, SerNet GmbH https://sernet.de/en/samba/
More information about the samba-technical
mailing list