jmeter-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ham1 <...@git.apache.org>
Subject [GitHub] jmeter pull request #370: ConstantPoissonProcessGenerator sonar fix
Date Sun, 07 Jan 2018 17:02:29 GMT
GitHub user ham1 opened a pull request:

    https://github.com/apache/jmeter/pull/370

    ConstantPoissonProcessGenerator sonar fix

    Fix sonar 'bug'. It also makes it more obvious we are changing the `i` declared outside
the `do while` instead of redeclaring another `i` for the `for` loop.
    
    PS. If someone has a better name for this variable please change as the code isn't very
easy to understand, at least for me.


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/ham1/jmeter cppg_sonar_fix

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/jmeter/pull/370.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 #370
    
----
commit 9e584c4db43627e06df469b0cedd4ddee41f3594
Author: Graham Russell <graham@...>
Date:   2018-01-07T16:15:26Z

    fixed sonar warning

commit 7aeb30b9d8e44d990efb589fac3b24c1907829f4
Author: Graham Russell <graham@...>
Date:   2018-01-07T16:15:37Z

    formatting

----


---

Mime
View raw message