[PATCH] smbstatus doesn't show lease info

Jeremy Allison jra at samba.org
Mon Oct 12 19:53:35 UTC 2015


On Mon, Oct 12, 2015 at 09:45:20PM +0200, Ralph Boehme wrote:
> On Mon, Oct 12, 2015 at 09:59:05AM -0700, Jeremy Allison wrote:
> > On Mon, Oct 12, 2015 at 01:17:59PM +0200, Ralph Boehme wrote:
> > > Hi,
> > > 
> > > attached is a fix for bug 11549.
> > > 
> > > Please review & push if ok. Thanks!
> > 
> > I think you should have a helper variable
> > here to make it more efficient (this is used
> > in may other places in the code).
> > 
> > How about the attached instead ?
> 
> sure, if you prefer. But I doubt this makes a difference for a decent
> optimizing compiler.

I used to believe that too... But lots of Volkers
experiments have convinced me otherwise :-).

Thanks - I'll push !



More information about the samba-technical mailing list