[GitHub] [jmeter] vlsi opened a new pull request, #6011: fix: default value for LoopController.continue_forever rather than initializing in the constructor

["vlsi (via GitHub)" <gi...@apache.org>]

vlsi opened a new pull request, #6011:
URL: https://github.com/apache/jmeter/pull/6011

   Previously LoopController initialized continue_forever=true in the constructor, however, and ThreadGroup.setMainController flipped the value to false.
   
   It caused inconsistency since non-GUI launch does not call ThreadGroup.setMainController.
   
   Using default=true enables to treat "false" as non-standard, so =false is saved t the jmx, and it gets loaded in non-GUI as well.
   
   Fixes https://github.com/apache/jmeter/issues/6008
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@jmeter.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [jmeter] vlsi merged pull request #6011: fix: use default value for LoopController.continue_forever rather than initializing it in the constructor

["vlsi (via GitHub)" <gi...@apache.org>]

vlsi merged PR #6011:
URL: https://github.com/apache/jmeter/pull/6011


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@jmeter.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org