Throw more informative error if MPI.Init() is called multiple times #310

simonbyrne · 2019-09-23T22:30:44Z

Fixes #309.

lcw · 2019-09-23T22:57:50Z

I just tried this in the repl and it seems to work fine!

simonbyrne · 2019-09-23T22:59:40Z

I just tried this in the repl and it seems to work fine!

What do you mean? That calling MPI.Init(); MPI.Finalize(); MPI.Init() worked, or it threw the correct error?

lcw · 2019-09-23T23:03:59Z

What do you mean? That calling `MPI.Init(); MPI.Finalize(); MPI.Init()` worked, or it threw the correct error?

Sorry to be unclear. Threw the correct error. I suggest the change be merged.

simonbyrne · 2019-09-24T04:57:35Z

Cheers, thanks @lcw.

Throw more informative error if MPI.Init() is called multiple times

9c69857

Fixes #309.

simonbyrne merged commit edab8ef into master Sep 24, 2019

simonbyrne deleted the sb/init-state branch September 24, 2019 04:57

Provide feedback