[GitHub] [jmeter] FSchumacher opened a new pull request, #5874: Trim variable names in Argument objects

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

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

   ## Description
   
   Trim names in Argument objects.
   
   ## Motivation and Context
   
   Should fix #5872
   
   I think, we never intended to allow whitespace at the start or ending of a variable name.
   
   ## How Has This Been Tested?
   
   Added a JUnit test case for `Argumet#setName(String)`
   
   ## Screenshots (if appropriate):
   
   ## Types of changes
   
   - Bug fix (non-breaking change which fixes an issue)
   
   ## Checklist:
   
   - [x] My code follows the [code style][style-guide] of this project.
   - [ ] I have updated the documentation accordingly.
   
   [style-guide]: https://wiki.apache.org/jmeter/CodeStyleGuidelines
   


-- 
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] FSchumacher merged pull request #5874: Trim variable names in Argument objects

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

FSchumacher merged PR #5874:
URL: https://github.com/apache/jmeter/pull/5874


-- 
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