You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@jmeter.apache.org by sebb <se...@gmail.com> on 2012/06/06 23:25:00 UTC

Re: svn commit: r1347105 - /jmeter/trunk/src/core/org/apache/jmeter/visualizers/RunningSample.java

On 6 June 2012 21:39,  <pm...@apache.org> wrote:
> Author: pmouawad
> Date: Wed Jun  6 20:39:16 2012
> New Revision: 1347105
>
> URL: http://svn.apache.org/viewvc?rev=1347105&view=rev
> Log:
> DecimalFormat is not thread-safe, so remove static instance

Good catch.

> Modified:
>    jmeter/trunk/src/core/org/apache/jmeter/visualizers/RunningSample.java
>
> Modified: jmeter/trunk/src/core/org/apache/jmeter/visualizers/RunningSample.java
> URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/visualizers/RunningSample.java?rev=1347105&r1=1347104&r2=1347105&view=diff
> ==============================================================================
> --- jmeter/trunk/src/core/org/apache/jmeter/visualizers/RunningSample.java (original)
> +++ jmeter/trunk/src/core/org/apache/jmeter/visualizers/RunningSample.java Wed Jun  6 20:39:16 2012
> @@ -38,9 +38,9 @@ import org.apache.jmeter.samplers.Sample
>  */
>  public class RunningSample {
>
> -    private static final DecimalFormat rateFormatter = new DecimalFormat("#.0"); // $NON-NLS-1$
> +    private final DecimalFormat rateFormatter = new DecimalFormat("#.0"); // $NON-NLS-1$
>
> -    private static final DecimalFormat errorFormatter = new DecimalFormat("#0.00%"); // $NON-NLS-1$
> +    private final DecimalFormat errorFormatter = new DecimalFormat("#0.00%"); // $NON-NLS-1$
>
>     private long counter;
>
>
>