Remove legacy z_config_peer/z_config_client methods

CMakeLists.txt

@@ -414,7 +414,6 @@ if(UNIX OR MSVC)
     add_executable(z_bytes_test ${PROJECT_SOURCE_DIR}/tests/z_bytes_test.c)
     add_executable(z_api_bytes_test ${PROJECT_SOURCE_DIR}/tests/z_api_bytes_test.c)
     add_executable(z_api_encoding_test ${PROJECT_SOURCE_DIR}/tests/z_api_encoding_test.c)
-    add_executable(z_api_config_test ${PROJECT_SOURCE_DIR}/tests/z_api_config_test.c)
     add_executable(z_refcount_test ${PROJECT_SOURCE_DIR}/tests/z_refcount_test.c)
 
     target_link_libraries(z_data_struct_test ${Libname})
@@ -433,7 +432,6 @@ if(UNIX OR MSVC)
     target_link_libraries(z_bytes_test ${Libname})
     target_link_libraries(z_api_bytes_test ${Libname})
     target_link_libraries(z_api_encoding_test ${Libname})
-    target_link_libraries(z_api_config_test ${Libname})
     target_link_libraries(z_refcount_test ${Libname})
 
     configure_file(${PROJECT_SOURCE_DIR}/tests/modularity.py ${CMAKE_RUNTIME_OUTPUT_DIRECTORY}/modularity.py COPYONLY)
@@ -456,7 +454,6 @@ if(UNIX OR MSVC)
     add_test(z_bytes_test ${CMAKE_RUNTIME_OUTPUT_DIRECTORY}/z_bytes_test)
     add_test(z_api_bytes_test ${CMAKE_RUNTIME_OUTPUT_DIRECTORY}/z_api_bytes_test)
     add_test(z_api_encoding_test ${CMAKE_RUNTIME_OUTPUT_DIRECTORY}/z_api_encoding_test)
-    add_test(z_api_config_test ${CMAKE_RUNTIME_OUTPUT_DIRECTORY}/z_api_config_test)
     add_test(z_refcount_test ${CMAKE_RUNTIME_OUTPUT_DIRECTORY}/z_refcount_test)
   endif()
 

docs/api.rst

@@ -304,8 +304,6 @@ Primitives
 .. autocfunction:: primitives.h::z_keyexpr_join
 .. autocfunction:: primitives.h::z_config_new
 .. autocfunction:: primitives.h::z_config_default
-.. autocfunction:: primitives.h::z_config_client
-.. autocfunction:: primitives.h::z_config_peer
 .. autocfunction:: primitives.h::zp_config_get
 .. autocfunction:: primitives.h::zp_config_insert
 .. autocfunction:: primitives.h::z_encoding_from_str

include/zenoh-pico/api/primitives.h

@@ -309,29 +309,6 @@ int8_t z_config_default(z_owned_config_t *config);
  */
 void z_config_clone(z_owned_config_t *config, const z_loaned_config_t *src);
 
-/**
- * Builds a new, zenoh-allocated, client configuration.
- *
- * Parameters:
- *   config: Pointer to uninitialized :c:type:`z_owned_config_t`.
- *   locator: Zenoh router locator string, if null multicast scouting will be performed.
- *
- * Return:
- *   `0`` in case of success, or a ``negative value`` otherwise.
- */
-int8_t z_config_client(z_owned_config_t *config, const char *locator);
-/**
- * Builds a new, zenoh-allocated, peer configuration.
- *
- * Parameters:
- *   config: Pointer to uninitialized :c:type:`z_owned_config_t`.
- *   locator: Multicast address for peer-to-peer communication.
- *
- * Return:
- *   `0`` in case of success, or a ``negative value`` otherwise.
- */
-int8_t z_config_peer(z_owned_config_t *config, const char *locator);
-
 /**
  * Gets the property with the given integer key from the configuration.
  *

src/api/api.c

@@ -190,22 +190,6 @@ int8_t z_config_default(z_owned_config_t *config) { return _z_config_default(&co
 
 void z_config_clone(z_owned_config_t *config, const z_loaned_config_t *src) { config->_val = _z_config_clone(src); }
 
-int8_t z_config_client(z_owned_config_t *config, const char *locator) {
-    return _z_config_client(&config->_val, locator);
-}
-
-int8_t z_config_peer(z_owned_config_t *config, const char *locator) {
-    if (locator == NULL) {
-        return _Z_ERR_INVALID;
-    }
-    _Z_RETURN_IF_ERR(z_config_default(config));
-    _Z_CLEAN_RETURN_IF_ERR(zp_config_insert(&config->_val, Z_CONFIG_MODE_KEY, Z_CONFIG_MODE_PEER),
-                           z_config_drop(z_config_move(config)));
-    _Z_CLEAN_RETURN_IF_ERR(zp_config_insert(&config->_val, Z_CONFIG_CONNECT_KEY, locator),
-                           z_config_drop(z_config_move(config)));
-    return _Z_RES_OK;
-}
-
 const char *zp_config_get(const z_loaned_config_t *config, uint8_t key) { return _z_config_get(config, key); }
 
 int8_t zp_config_insert(z_loaned_config_t *config, uint8_t key, const char *value) {