[PATCHS] make pthreadpool common
Jeremy Allison
jra at samba.org
Thu Feb 9 18:49:49 UTC 2017
On Thu, Feb 09, 2017 at 09:26:26AM +0100, Volker Lendecke wrote:
> On Wed, Feb 08, 2017 at 02:46:55PM -0800, Matthieu Patou wrote:
> >
> > We discussed this last week, there is a great piece of work in
> > source3/lib/pthreadpool and it just seems that it could be used by more
> > components than winbindd and the file server. So I'm proposing with
> > those patches to move it to the top of tree.
> >
> > I did a build and also autobuild (which failed after 2h5 minutes in some
> > AD DC DNS related tests, so I don't think it related to my change).
> >
> >
> > Please review !
>
> I'd like the comment in the last patch to slightly change: I don't think
> that it is a requirement for the usability of some component to live
> in the toplevel. I'd have to look, but I'm 100% certain we have
> toplevel and/or AD DC components depending on code in source3 and we
> have compontents in source3 that depend on code in source4. So this is
> a purely cosmetic change.
Pushed, after changing the comment to : "Move pthreadpool to top of the tree."
I agree this is comsmetic, but it really does help to re-arrange
the code in this way.
New programmers to Samba will now easily understand that the
pthreadpool implementation is a generic component we should
encourage use of in new code, not a smbd/Samba3-subsystem
specific library.
It does (IMHO) make our codebase a lot more friendly for
new developers to understand.
Jeremy.
More information about the samba-technical
mailing list