zc_shm_client_list_add_client should accept the list as first parameter

[milyin]

Describe the release item

This function changes the list. I.e. it's kind of method of the list object and should accept list as first parameter.
It's necessary also to check other functions for arguments order.

pub extern "C" fn zc_shm_client_list_add_client(
    id: z_protocol_id_t,
    client: &mut z_owned_shm_client_t,
    list: &mut zc_loaned_shm_client_list_t,
) -> z_error_t {
    match client.as_rust_type_mut().take() {
        Some(client) => {
            list.as_rust_type_mut().push((id, client));
            Z_OK
        }
        None => Z_EINVAL,
    }
}

[milyin]

This function itself is not important as it's shm and so unsable. But setting priority to high as we need to make review of other constructor function for parameters order

[milyin]

reviewed API, result in issue #647, closing task