[GitHub] jmeter pull request #404: Use log string format instead of doing the concate...

2018-10-12 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/jmeter/pull/404


---


[GitHub] jmeter pull request #404: Use log string format instead of doing the concate...

2018-10-10 Thread FSchumacher
GitHub user FSchumacher opened a pull request:

https://github.com/apache/jmeter/pull/404

Use log string format instead of doing the concatenation ourselves

## Description
Simplify code.

## Motivation and Context
Make code a bit more readable by using placeholders for the
parameters. We can get rid of a few `if` statements, where
parameters are used directly in the log statements.

## How Has This Been Tested?
No functional change.

## Screenshots (if appropriate):

## Types of changes
- code cleanup

## Checklist:


- [x] My code follows the [code style][style-guide] of this project.

[style-guide]: https://wiki.apache.org/jmeter/CodeStyleGuidelines


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/FSchumacher/jmeter use-log-string-format

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/jmeter/pull/404.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #404


commit bf424d461e4fc16d2142865f92bbf8ac0e72c807
Author: Felix Schumacher 
Date:   2018-10-10T20:21:43Z

Use log string format instead of doing the concatenation ourselves

Make code a bit more readable by using placeholders for the
parameters. We can get rid of a few `if` statements, where
parameters are used directly in the log statements.




---