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 less 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