sabi0 opened a new pull request, #13068:
URL: https://github.com/apache/lucene/pull/13068
   I noticed there are `brToString()` methods in 16 different classes.
   What do you think of providing a shared implementation in the 
`ToStringUtils` class?
   
   If you find the proposal useful I will proceed with removing the other 
copies.
   
   Otherwise the `ToStringUtils` class could probably be deleted - the two 
methods there `byteArray()` and `longHex()` are not used anywhere.


-- 
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: issues-unsubscr...@lucene.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org

Reply via email to