On 13 December 2016 at 09:20, max3163 <g...@git.apache.org> wrote:
> Github user max3163 commented on a diff in the pull request:
>
>     https://github.com/apache/jmeter/pull/238#discussion_r92125661
>
>     --- Diff: src/core/org/apache/jmeter/threads/ThreadGroup.java ---
>     @@ -279,34 +282,30 @@ private void delayBy(long delay) {
>          @Override
>          public void start(int groupCount, ListenerNotifier notifier, 
> ListedHashTree threadGroupTree, StandardJMeterEngine engine) {
>              running = true;
>     +        numGroup = groupCount;

Does numGroup mean:
- the total number of groups
- the number of this group

IMO groupCount can only mean the count (i.e. number) of groups so is
less ambiguous.

>     --- End diff --
>
>     It has better meaning
>
>
> ---
> 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 infrastruct...@apache.org or file a JIRA ticket
> with INFRA.
> ---

Reply via email to