You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jmeter.apache.org by fs...@apache.org on 2018/11/04 11:37:55 UTC

svn commit: r1845725 - in /jmeter/trunk: src/core/org/apache/jmeter/threads/JMeterContext.java xdocs/changes.xml xdocs/usermanual/functions.xml

Author: fschumacher
Date: Sun Nov  4 11:37:54 2018
New Revision: 1845725

URL: http://svn.apache.org/viewvc?rev=1845725&view=rev
Log:
Add location of JMeterContext API to link and write out numbers as words

Follow-up to r1845700
Bug 62752 - Add to Documentation: ctx.getThreadNum() 0-based and
${__threadNum} 1-based

Bugzilla Id: 62752


Modified:
    jmeter/trunk/src/core/org/apache/jmeter/threads/JMeterContext.java
    jmeter/trunk/xdocs/changes.xml
    jmeter/trunk/xdocs/usermanual/functions.xml

Modified: jmeter/trunk/src/core/org/apache/jmeter/threads/JMeterContext.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/threads/JMeterContext.java?rev=1845725&r1=1845724&r2=1845725&view=diff
==============================================================================
--- jmeter/trunk/src/core/org/apache/jmeter/threads/JMeterContext.java (original)
+++ jmeter/trunk/src/core/org/apache/jmeter/threads/JMeterContext.java Sun Nov  4 11:37:54 2018
@@ -143,9 +143,7 @@ public class JMeterContext {
     }
 
     /**
-     * Returns the threadNum which starts at 0
-     *
-     * @return int
+     * @return the threadNum starting from one
      */
     public int getThreadNum() {
         return threadNum;

Modified: jmeter/trunk/xdocs/changes.xml
URL: http://svn.apache.org/viewvc/jmeter/trunk/xdocs/changes.xml?rev=1845725&r1=1845724&r2=1845725&view=diff
==============================================================================
--- jmeter/trunk/xdocs/changes.xml [utf-8] (original)
+++ jmeter/trunk/xdocs/changes.xml [utf-8] Sun Nov  4 11:37:54 2018
@@ -116,7 +116,7 @@ Summary
   <li><bug>62826</bug>When changing LAF, make JMeter restart if user clicks yes to popup</li>
   <li><bug>62744</bug>Upgrade jquery to version 3.3.1, jquery-ui to 1.12.1, bootstrap to 3.3.7</li>
    <li><bug>62257</bug><pr>401</pr>Expand/Collapse short key <keysym>-</keysym> (minus sign) on numpad doesn't work. Contributed by Ori Marko (orimarko at gmail.com)</li>
-   <li><bug>62752</bug>Add to Documentation: <code>ctx.getThreadNum()</code> is 0-based while <code>${__threadNum}</code> is 1-based</li>
+   <li><bug>62752</bug>Add to Documentation: <code>ctx.getThreadNum()</code> is zero-based while <code>${__threadNum}</code> is one-based</li>
 </ul>
 
 <ch_section>Non-functional changes</ch_section>

Modified: jmeter/trunk/xdocs/usermanual/functions.xml
URL: http://svn.apache.org/viewvc/jmeter/trunk/xdocs/usermanual/functions.xml?rev=1845725&r1=1845724&r2=1845725&view=diff
==============================================================================
--- jmeter/trunk/xdocs/usermanual/functions.xml (original)
+++ jmeter/trunk/xdocs/usermanual/functions.xml Sun Nov  4 11:37:54 2018
@@ -378,8 +378,9 @@ If you want to have a count that increme
 <description><p>The thread number function simply returns the number of the thread currently
 being executed.  These numbers are independent of ThreadGroup, meaning thread #1 in one threadgroup
 is indistinguishable from thread #1 in another threadgroup, from the point of view of this function.</p>
-<note>The function returns a number between 1 and the max number of running threads. Note that if you're using 
-JSR223 code with <a href="" >JMeterContext</a> object (ctx variable), the below code returns a number between 0 and (max number of running threads - 1)
+<note>The function returns a number between one and the max number of running threads. Note that if you're using
+JSR223 code with <a href="../api/org/apache/jmeter/threads/JMeterContext.html">JMeterContext</a> object (<code>ctx</code> variable),
+the below code returns a number between zero and (max number of running threads minus one)
 <source>ctx.getThreadNum()</source>
 </note>