You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@logging.apache.org by GitBox <gi...@apache.org> on 2021/06/18 21:19:24 UTC
[GitHub] [logging-log4j2] arturobernalg opened a new pull request #525: Replace null check with Objects/Stream static method.
arturobernalg opened a new pull request #525:
URL: https://github.com/apache/logging-log4j2/pull/525
--
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
[GitHub] [logging-log4j2] vy merged pull request #525: Replace null check with Objects/Stream static method.
Posted by GitBox <gi...@apache.org>.
vy merged pull request #525:
URL: https://github.com/apache/logging-log4j2/pull/525
--
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
[GitHub] [logging-log4j2] arturobernalg commented on pull request #525: Replace null check with Objects/Stream static method.
Posted by GitBox <gi...@apache.org>.
arturobernalg commented on pull request #525:
URL: https://github.com/apache/logging-log4j2/pull/525#issuecomment-864242637
> While I don't see anything wrong with this, this is another case where I don't really find any advantage to the new code.
HI @rgoers
IMO it's more readable
TY
--
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
[GitHub] [logging-log4j2] rgoers commented on pull request #525: Replace null check with Objects/Stream static method.
Posted by GitBox <gi...@apache.org>.
rgoers commented on pull request #525:
URL: https://github.com/apache/logging-log4j2/pull/525#issuecomment-864159417
While I don't see anything wrong with this, this is another case where I don't really find any advantage to the new code.
--
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