[PATCH] Next round of netlogon_cli_creds refactoring

[Volker Lendecke]

On Mon, Sep 25, 2017 at 01:33:15PM +1300, Andrew Bartlett wrote:
> The final patch looks wrong, was it brought in from a larger branch you
> were working on?  A compiler with --picky-developer shows that with
> netlogon_creds_cli_lock() static, it is un-used.

No, this was deliberate. I want to get rid of the cli_lock function. I
just had not run a full autobuild with this, functionally making a
function static should not make any difference. Without that, it had
survived an autobuild. It's only the sync wrapper that is unused, the
async ones are still used but should not be used outside this
function.

> Otherwise it seems OK to me.

Is that a formal RB+?

Thanks, Volker