[Fwd: Re: [PATCH] Minor cleanup of libnet_LookupName_recv] || 2nd reviewer, please

Christof Schmitt cs at samba.org
Fri Mar 2 16:25:11 UTC 2018


On Wed, Feb 28, 2018 at 03:46:49PM -0700, Christof Schmitt via samba-technical wrote:
> On Tue, Feb 27, 2018 at 03:21:14PM +0100, Ralph Böhme via samba-technical wrote:
> > On Tue, Feb 27, 2018 at 03:13:47PM +0100, Ralph Böhme via samba-technical wrote:
> > > On Mon, Feb 26, 2018 at 04:52:16PM +0100, Swen Schillig wrote:
> > > > On Mon, 2018-02-26 at 16:47 +0100, Ralph Böhme wrote:
> > > > > Hi Swen,
> > > > > 
> > > > > On Mon, Feb 26, 2018 at 04:17:38PM +0100, Swen Schillig via samba-
> > > > > technical wrote:
> > > > > > Anyway, as I said above, I have no strong feelings about using or
> > > > > > not
> > > > > > using "unlikely" here. I f the two of you prefer, I will remove it.
> > > > > 
> > > > > yes, please submit an updated patchset without the unlikely and I will be
> > > > > happy to re-review. :)
> > > >
> > > > Sure, here we go.
> > > 
> > > thanks! Pushed.
> > 
> > ups, cancelled. Christof, can you please re-review the (slightly) updated
> > patchset? Thanks.
> 
> lgtm

Pushed to autobuild with Ralph's and my Reviewed-by.

Christof



More information about the samba-technical mailing list