Review request: CASSANDRA-13799

[Malcolm Taylor <ma...@semmle.com>]

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 cause confusion. All the fixes are very small changes, so it
should be relatively easy to review. I have only generated a patch for
trunk, because that is the branch that is analysed by lgtm.com, but please
let me know if I should provide additional information.
Thanks,
Malcolm

Re: Review request: CASSANDRA-13799

[Jeff Jirsa <jj...@apache.org>]

On 2017-09-06 00:58, Malcolm Taylor <ma...@semmle.com> 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 such as renaming a variable where the current
> name may cause confusion. All the fixes are very small changes, so it
> should be relatively easy to review. I have only generated a patch for
> trunk, because that is the branch that is analysed by lgtm.com, but please
> let me know if I should provide additional information.
> Thanks,
> Malcolm
> 

I'll try to take it soon'ish.



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@cassandra.apache.org
For additional commands, e-mail: dev-help@cassandra.apache.org