[PATCH] Finish multi-process netlogon server for ncacn_ip_tcp

Volker Lendecke Volker.Lendecke at SerNet.DE
Thu Jul 27 06:16:15 UTC 2017


On Thu, Jul 27, 2017 at 03:06:25PM +1200, Andrew Bartlett via samba-technical wrote:
> The attached patches Douglas has reviewed and I've pushed to autobuild,
> but I pass them here for transparency. 

Please make sure that no lines exceed 80 characters. There's a snippet
in our README.Coding that I paste here for reference:

> * Maximum Line Width is 80 Characters
>   The reason is not about people with low-res screens but rather sticking
>   to 80 columns prevents you from easily nesting more than one level of
>   if statements or other code blocks.  Use source3/script/count_80_col.pl
>   to check your changes.

Also, if you use DBG_NOTICE instead of DEBUG(4, you get a shorter line
for free.

Thanks,

Volker

-- 
SerNet GmbH, Bahnhofsallee 1b, 37081 Göttingen
phone: +49-551-370000-0, fax: +49-551-370000-9
AG Göttingen, HRB 2816, GF: Dr. Johannes Loxen
http://www.sernet.de, mailto:kontakt at sernet.de



More information about the samba-technical mailing list