On 26/05/2020 11:22, Aleks Efimov wrote:
Hi Alan,

Thank you for the review. I've renamed 'appendAddresses' to 'concatAddresses' per your suggestion. And also removedĀ  'arrangeAddresses' method and moved its code to 'lookupAllHostAddr'.

Webrev with new version can be viewed here:
http://cr.openjdk.java.net/~aefimov/8244958/02
Thanks, this looks okay to me (I agree with Daniel that isEmpty is a bit better).

-Alan

Reply via email to