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 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-unsubscr...@cassandra.apache.org
For additional commands, e-mail: dev-h...@cassandra.apache.org



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