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

fidencio commented:
"""
Personally I think any of the suggestions make the code more readable than the 
switch (which as all the cases explicitly matching with the comments above).

For me, it's a nack for the patch and for those suggestions.
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/250#issuecomment-297676645
_______________________________________________
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