Hi Felix, Reverted , can you explain to me this ? I don't see these fields used, are they through reflection ?
Thanks On Sun, Jan 25, 2015 at 10:30 PM, Felix Schumacher < [email protected]> wrote: > > > Am 25. Januar 2015 21:35:48 MEZ, schrieb [email protected]: > >Author: pmouawad > >Date: Sun Jan 25 20:35:48 2015 > >New Revision: 1654701 > > > >URL: http://svn.apache.org/r1654701 > >Log: > >Remove unused fields > > > >Modified: > > jmeter/trunk/src/core/org/apache/jmeter/util/ThreadLocalRandom.java > > > >Modified: > >jmeter/trunk/src/core/org/apache/jmeter/util/ThreadLocalRandom.java > >URL: > > > http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/util/ThreadLocalRandom.java?rev=1654701&r1=1654700&r2=1654701&view=diff > > >============================================================================== > >--- jmeter/trunk/src/core/org/apache/jmeter/util/ThreadLocalRandom.java > >(original) > >+++ jmeter/trunk/src/core/org/apache/jmeter/util/ThreadLocalRandom.java > >Sun Jan 25 20:35:48 2015 > >@@ -54,11 +54,6 @@ public class ThreadLocalRandom extends R > > */ > > boolean initialized; > > > >- // Padding to help avoid memory contention among seed updates in > >- // different TLRs in the common case that they are located near > >- // each other. > >- private long pad0, pad1, pad2, pad3, pad4, pad5, pad6, pad7; > > Given the comment above the fields, I would not remove those padding > fields. > > Regards > Felix > >- > > /** > > * The actual ThreadLocal > > */ > > -- Cordialement. Philippe Mouawad.
