On 20 September 2014 04:26, martinschaef <[email protected]> wrote:
> GitHub user martinschaef opened a pull request:
>
>     https://github.com/apache/jmeter/pull/10
>
>     Fixed two minor things found by static analysis
>
>
>
> You can merge this pull request into a Git repository by running:
>
>     $ git pull https://github.com/martinschaef/jmeter trunk
>
> Alternatively you can review and apply these changes as the patch at:
>
>     https://github.com/apache/jmeter/pull/10.patch
>
> To close this pull request, make a commit to your master/trunk branch
> with (at least) the following in the commit message:
>
>     This closes #10
>
> ----
> commit 6f4c187457f0cd0d7325d5260fe634edac9b5a98
> Author: Martin <[email protected]>
> Date:   2014-09-20T03:22:32Z
>
>     Removed duplicated check

That is OK.

> commit 99d80617e2263d097e70426d83c7ac711b17c0c6
> Author: Martin <[email protected]>
> Date:   2014-09-20T03:25:10Z
>
>     Removed trivial condition

False positive in static analysis; current[0] can be changed by the
read(current) invocation.

> ----
>
>
> ---
> If your project is set up for it, you can reply to this email and have your
> reply appear on GitHub as well. If your project does not have this feature
> enabled and wishes so, or if the feature is enabled but not working, please
> contact infrastructure at [email protected] or file a JIRA ticket
> with INFRA.
> ---

Reply via email to