Re: Review request: CASSANDRA-13799

2017-09-15 Thread Jeff Jirsa
On 2017-09-06 00:58, Malcolm Taylor wrote: > Can someone please review my patch on CASSANDRA-13799 ? The issue fixes > several alerts raised by lgtm.com, which runs a deep analysis of the code > in trunk. Some of the alerts refer to clear bugs, while others are > straightforward improvements su

Review request: CASSANDRA-13799

2017-09-06 Thread Malcolm Taylor
Can someone please review my patch on CASSANDRA-13799 ? The issue fixes several alerts raised by lgtm.com, which runs a deep analysis of the code in trunk. Some of the alerts refer to clear bugs, while others are straightforward improvements such as renaming a variable where the current name may ca