kenliao94 commented on PR #1222:
URL: https://github.com/apache/activemq/pull/1222#issuecomment-2108232172

   > There are a bunch of varying changes here. Not sure it makes sense as a 
single commit. Perhaps the iteration Java-language updates go as one easy 
commit.
   > 
   > I do not see a reason to remove null checks.
   > 
   > Is this PR part of a JIRA or NO-JIRA change?
   
   This is part of a NO-JIRA change. I am just trying to clean up the code as I 
navigate the code :) I moved the removing redundant null checks from this PR to 
another I will create later to make this PR more about trivial updates. 


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to