URL: https://github.com/SSSD/sssd/pull/250
Title: #250: cache_req: Use ternary operator instead of bool+switch

lslebodn commented:
"""
>I Personally I think any of the suggestions make the code less readable than 
>the switch (which as all the cases explicitly matching with the comments 
>above).
I'm sorry but switch is the worst option from proposed.

It looks like 2 guys like version with `if`. I will prepare version with `if` 
even though that ternary is more readable :-). Democracy works.
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/250#issuecomment-297701355
_______________________________________________
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org

Reply via email to