[PATCH] Do not reset the secrets.tdb SID and GUID when not required
Volker Lendecke
Volker.Lendecke at SerNet.DE
Wed May 21 05:56:49 MDT 2014
On Wed, May 21, 2014 at 01:49:53PM +0200, Andreas Schneider wrote:
> Some time ago I summarized our coding style rules and the ideas behind it in a
> blog post:
>
> https://blog.cryptomilk.org/2013/03/28/writing-and-reading-code/
>
> So 'if (!dom_sid_equal())' is by our coding style definition allowed but the
> extra bool ok is appreciated :)
By that blog post (ok == false) is clearer. Quote:
> if (c == false) {
> return;
> }
This comes with another Quote:
> If you look at the code now, you can probably guess what
> types they are.
So my question remains: Do we in the future forbid !ok and mandate
ok==false, such that the type of "ok" becomes cleaner?
With best regards,
Volker Lendecke
--
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