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