URL: https://github.com/SSSD/sssd/pull/838
Title: #838: FIPS140 compliant usage of PRNG

alexey-tikhonov commented:
"""
> > > 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 
https://github.com/SSSD/sssd/pull/838#issuecomment-509556008
_______________________________________________
sssd-devel mailing list -- [email protected]
To unsubscribe send an email to [email protected]
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedorahosted.org/archives/list/[email protected]

Reply via email to