[PATCH] Enhance vfs_fruit manpage and testparm checks for vfs_fruit stuff

Jeremy Allison jra at samba.org
Wed Mar 9 17:21:26 UTC 2016


On Wed, Mar 09, 2016 at 06:04:07PM +0100, Ralph Boehme wrote:
> On Wed, Mar 09, 2016 at 08:47:26AM -0800, Jeremy Allison wrote:
> > On Wed, Mar 09, 2016 at 05:19:05PM +0100, Ralph Boehme wrote:
> > > Hi!
> > > 
> > > Attached is a patchset that adds some important information to the
> > > vfs_fruit manpage and adds checks to testparm that cover this.
> > > 
> > > Please review & push if ok. Thanks!
> > 
> > LGTM. Pushed.
> 
> thanks!
> 
> > Minor complain. I *hate* new static's, although I can
> > see why you used them :-).
> > 
> > Lemme think if there's a better way (can't see one
> > right now though...).
> 
> create a state in the caller and pass it to the function and then do
> the check in the caller, not the function. But is that really
> necessary for testparm?

Yeah, that's what I thought too. But came to the
same conclusion...

But are we just being lazy and incurring a technical
debt we'll have to pay later ?

:-).



More information about the samba-technical mailing list