You are viewing a plain text version of this content. The canonical link for it is here.

Posted to gitbox@activemq.apache.org by GitBox <gi...@apache.org> on 2022/02/08 22:46:49 UTC

[GitHub] [activemq] mattrpav commented on pull request #760: Some issues from static code analysis

mattrpav commented on pull request #760:
URL: https://github.com/apache/activemq/pull/760#issuecomment-1033138273


   This PR is not commit-able. Static code analysis tools work well for end-developer, but don't handle the context of infrastructure software such as messaging brokers and databases.
   
   1. Too many changes in on PR
   2. Missing context-- catching 'Throwable' is by design in many places
   3. This PR adds 'boxed' primitives back in
   4. other stuff.


-- 
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: gitbox-unsubscribe@activemq.apache.org

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