> > there are multiple new lines around sss_rand() functions,
only one empty line should be there
>
> I changed this as well, but actually I was unable to find such a requirement in
Coding guidelines.
> Do you think guide should be updated?
Well, it's not explicitly written but it is shown:
https://docs.pagure.org/SSSD.sssd/developers/coding_style.html#if-statements
This is a good example of braces placement rules.
But I hardly can understand this as an example of "newlines rule".
"""
See the full comment at