jmeter-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [jmeter] FSchumacher opened a new pull request #626: Move guard clause for log message generation
Date Sat, 26 Sep 2020 12:21:48 GMT

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


   
   ## Description
   Move guard close for log message generation a bit closer to the line, where the logging
is done.
   
   ## Motivation and Context
   The guard clause is not visible for sonaqube and we
   can silence a warning by this move. Another reason
   for the move is, that we save a few lines of code.
   
   ## How Has This Been Tested?
   Let the ci test it.
   
   ## Screenshots (if appropriate):
   
   ## Types of changes
   - Code style
   
   ## Checklist:
   <!--- Go over all the following points, and put an `x` in all the boxes that apply.
-->
   <!--- If you're unsure about any of these, don't hesitate to ask. We're here to help!
-->
   - [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.

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



Mime
View raw message