Rename Endpoint methods for consistency

Author: gavv

roc_jni/src/main/export/org_rocstreaming_roctoolkit_Endpoint.h

@@ -134,7 +134,7 @@ void endpoint_set_resource(JNIEnv* env, jobject endpoint, const char* buf) {
     (*env)->SetObjectField(env, endpoint, attrId, (*env)->NewStringUTF(env, buf));
 }
 
-JNIEXPORT void JNICALL Java_org_rocstreaming_roctoolkit_Endpoint_init(
+JNIEXPORT void JNICALL Java_org_rocstreaming_roctoolkit_Endpoint_nativeParseUri(
     JNIEnv* env, jobject thisObj, jstring juri) {
     jclass endpointClass = NULL;
     roc_endpoint* endpoint = NULL;
@@ -226,7 +226,7 @@ JNIEXPORT void JNICALL Java_org_rocstreaming_roctoolkit_Endpoint_init(
     }
 }
 
-JNIEXPORT jstring JNICALL Java_org_rocstreaming_roctoolkit_Endpoint_getUri(
+JNIEXPORT jstring JNICALL Java_org_rocstreaming_roctoolkit_Endpoint_nativeFormatUri(
     JNIEnv* env, jobject thisObj) {
     roc_endpoint* endpoint = NULL;
     jstring juri = NULL;
@@ -263,7 +263,7 @@ JNIEXPORT jstring JNICALL Java_org_rocstreaming_roctoolkit_Endpoint_getUri(
     return juri;
 }
 
-JNIEXPORT void JNICALL Java_org_rocstreaming_roctoolkit_Endpoint_validate(
+JNIEXPORT void JNICALL Java_org_rocstreaming_roctoolkit_Endpoint_nativeValidate(
     JNIEnv* env, jobject thisObj) {
     roc_endpoint* endpoint = NULL;
     char* uri = NULL;

roc_jni/src/main/impl/endpoint.c

@@ -94,12 +94,14 @@ public class Endpoint {
     private String resource;
 
     /**
-     * Create endpoint from uri
+     * Create endpoint from URI
      *
-     * @param uri uri
+     * @param uri   URI to parse
+     *
+     * @throws IllegalArgumentException  if URI is invalid
      */
-    public Endpoint(String uri) {
-        init(uri);
+    public Endpoint(String uri) throws IllegalArgumentException {
+        nativeParseUri(uri);
     }
 
     /**
@@ -115,14 +117,16 @@ public Endpoint(String uri) {
      *                 <p>
      *                 If port is set to -1, the standard port for endpoint protocol is used. This is
      *                 allowed only if the protocol defines its standard port.
-     * @param resource resource nullable. Specifies percent-encoded path and query
+     * @param resource resource is nullable. Specifies percent-encoded path and query
+     *
+     * @throws IllegalArgumentException  if URI components don't form a valid URI
      */
-    public Endpoint(Protocol protocol, String host, int port, String resource) {
+    public Endpoint(Protocol protocol, String host, int port, String resource) throws IllegalArgumentException {
         this.protocol = Check.notNull(protocol, "protocol");
         this.host = Check.notEmpty(host, "host");
         this.port = port;
         this.resource = resource;
-        validate();
+        nativeValidate();
     }
 
     /**
@@ -138,19 +142,29 @@ public Endpoint(Protocol protocol, String host, int port, String resource) {
      *                 <p>
      *                 If port is set to -1, the standard port for endpoint protocol is used. This is
      *                 allowed only if the protocol defines its standard port.
+     *
+     * @throws IllegalArgumentException  if URI components don't form a valid URI
      */
     public Endpoint(Protocol protocol, String host, int port) {
         this(protocol, host, port, null);
     }
 
+    /**
+     * Get string URI describing this endpoint.
+     */
+    public String getUri() {
+        return nativeFormatUri();
+    }
+
+    /**
+     * Get string URI describing this endpoint.
+     */
     @Override
     public String toString() {
         return getUri();
     }
 
-    public native String getUri();
-
-    private native void init(String uri) throws IllegalArgumentException;
-
-    private native void validate() throws IllegalArgumentException;
+    private native void nativeParseUri(String uri) throws IllegalArgumentException;
+    private native String nativeFormatUri();
+    private native void nativeValidate() throws IllegalArgumentException;
 }