Github user mjeelanimsft commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/548#discussion_r203817750
--- Diff: src/java/main/org/apache/zookeeper/server/util/ConfigUtils.java
---
@@ -61,10 +61,16 @@ static public String getClientConfigStr(String
configData) {
return version + " " + sb.toString();
}
+ /**
+ * Splits server config to server and port
+ * with support for IPv6 literals
+ * @return String[] first element being the
+ * IP address and the next being the port
+ * @param s server config, server:port
+ */
public static String[] splitServerConfig(String s)
throws ConfigException
--- End diff --
Sure, I like that better too - I've renamed this
---
