On Wed, May 27, 2015 at 09:52:11AM +0200, Lukas Slebodnik wrote:
>+static errno_t
>+get_krb_primary(struct map_id_name_to_krb_primary *name_to_primary,
>+ char *id_prov_name, bool cs, const char **_krb_primary)
>+{
>+ errno_t ret;
>+ int i = 0;
>+
>+ while(name_to_primary != NULL &&
>+ name_to_primary[i].id_name != NULL &&
^^
I thought we have a convention to use
binary operatort in the begining of
line and not at the end.
IIRC Stephen sent mail with proposal and we agreed.
Sumit uses it quite often.
Is that a nack or a remark?
btw these proposals should be really codified in the coding style on
freeipa wiki (or our own coding style forked from freeipa).