Please use "error and out" logic (Re: [PATCH] Avoid privileged segfault in GetNCChanges and add many more tests)

[Simo]

On Wed, 2017-08-09 at 15:26 +0200, Ralph Böhme wrote:
> On Wed, Aug 09, 2017 at 09:10:22AM -0400, Simo via samba-technical
> wrote:
> > Thanks metze,
> > readability and modifiability of code are very important, and
> > become
> > even more important as a project grows and ages.
> > 
> > It would be nice to add a style section to the docs that more
> > forcefully and clearly explain these subtle coding style issues.
> 
> patch attached.
> 
> Cheerio!
> -slow

Reviewed-by: Simo Sorce <idra at samba.org>

Simo.