This repository has been archived by the owner on Mar 20, 2023. It is now read-only.

C++17: simplify dynamic MPI wrappers #837

Open

olupton opened this issue Jul 5, 2022 · 0 comments

Labels

Contributor

olupton commented Jul 5, 2022

In #834 we will require C++17. This will allow TODOs like

CoreNeuron/coreneuron/mpi/nrnmpi.h

Lines 71 to 73 in 0574812

    
           // This could be done with a simpler 
        
           //   template <auto fptr> struct function : function_base { ... }; 
        
           // pattern in C++17...

to be addressed, simplifying the code.

There is an internal reference to consult too.

olupton added the enhancement label

olupton mentioned this issue

Require C++17 #834

Merged

3 tasks

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.