Commit 78cade41 authored by Ramon Nou's avatar Ramon Nou
Browse files

Merge branch 'rnou/5-automatize-margo_register' into 'main' 

Resolve "Automatize MARGO_REGISTER"

Closes #5

See merge request !5
parents ae6722e7 ef692e86

src/network/engine.hpp

+15 −1
Original line number Diff line number Diff line
@@ -38,6 +38,19 @@ namespace scord::network { 


namespace detail {



#define REGISTER_RPC(__mid, __m_rpc_names, __func_name, __in_t, __out_t, __handler, requires_response) \

{ hg_id_t id = margo_provider_register_name(__mid, __func_name, \

    BOOST_PP_CAT(hg_proc_, __in_t), \

    BOOST_PP_CAT(hg_proc_, __out_t), \

    _handler_for_##__handler, \

    MARGO_DEFAULT_PROVIDER_ID, ABT_POOL_NULL); \

    __m_rpc_names.emplace(__func_name, id); \

    if(!requires_response) { \

            ::margo_registered_disable_response(__mid, id, HG_TRUE); \

       } \

}





struct margo_context {



    explicit margo_context(::margo_instance_id mid) : m_mid(mid) {}
@@ -90,7 +103,8 @@ struct engine { 
    register_rpcs() {



        // register RPCs manually for now

        m_context->register_rpc("ping", false);

        REGISTER_RPC(m_context->m_mid, m_context->m_rpc_names, "ping", void, void, ping, false);



    }



    void