Commit 0658d9bd authored by Alberto Miranda's avatar Alberto Miranda ♨️
Browse files

Merge branch... 

Merge branch 'amiranda/90-internal-adhoc_storage_info-should-offer-update-functions-to-simplify-code' into 'main'

Resolve "`internal::adhoc_storage_info` should offer `update` functions to simplify code"

Closes #90

See merge request !60
parents 1408d4f2 8d34ba9e

src/common/api/internal_types.hpp

+5 −0
Original line number Diff line number Diff line
@@ -73,6 +73,11 @@ struct adhoc_storage_info { 
        return m_adhoc_storage;

    }



    void

    update(admire::adhoc_storage::ctx adhoc_context) {

        m_adhoc_storage.update(std::move(adhoc_context));

    }



    admire::error_code

    add_client_info(std::shared_ptr<admire::internal::job_info> job_info) {

src/scord/adhoc_storage_manager.hpp

+1 −5
Original line number Diff line number Diff line
@@ -75,12 +75,8 @@ struct adhoc_storage_manager : scord::utils::singleton<adhoc_storage_manager> { 


        if(const auto it = m_adhoc_storages.find(id);

           it != m_adhoc_storages.end()) {



            const auto current_adhoc_info = it->second;

            auto tmp_adhoc = current_adhoc_info->adhoc_storage();

            tmp_adhoc.update(std::move(new_ctx));



            *it->second = admire::internal::adhoc_storage_info{tmp_adhoc};

            current_adhoc_info->update(std::move(new_ctx));

            return ADM_SUCCESS;

        }