modernize cabinet creation

change way vivaldi coordinates are managed internally

Upgrade embedded catch2 (-> v2.13.5).

Minor Sonar smells.

Const ref for std::string parameter.

Return const reference to vector.

Use a reference for ExecImpl::wait_any_for()'s vector.

Define CommImpl::test_any().

Simcall COMM_TESTANY is not blocking either. Simplify.

Define CommImpl::test().

Simcall COMM_TEST is not blocking.

Little simplification.

Finalize current CommImpl only once.

Homogenize debug messages across ActivityImpl's finish() methods.

Ensure there is only one simcall waiting.

[ci-skip] we want to order by the largest first ..

try to be preemptive about 32 bits complaints

this value was actually wrong, and caught by the new buffer overflow check.
this is due to the fact that actual mpich test uses 10 times higher values, so just removing last digit was breaking the BIGDFT=4*NUM_X-1 condition

adapt test for leak check

Add initialization/finalization check at each call.
Side effect was to break calls from external threads (gh-139).. So mark them as initialized properly at creation.

use previous buffer check feature in MPI checks, to crash when a buffer overflow is detected and provide good information.

Count all intercepted allocations/deallocations in order to build a small leak-checker.
This allows to check for leaks, but also to provide debug information on the involved buffers (file, line, size) when a crash happens
todo: evaluate memory cost and slowdown for real apps (disabled by default, needs --cfg=smpi/display-allocs or -analyze flag to be activated)

Use directly std::string instead of c_str.

s4u::Host::create_disk, human-friendly string version

s4u::Host: moving create_disk impl to HostImpl

s4u::Disk allow chaining set_property

s4u: allows full single-line initialization

seal() returns a pointer to the s4u object.
Allows users to use the object after its seal, without having to do it
in another line

Make fields private, and static functions class-member.

Useless forward declaration.

Redundant code (ctor is already inherited).

Prefer nullptr.

Kill erroneous constant.

Const/override for member functions.

Inherited constructor.

Pointer-to-const parameter.

continue to mess with platform creation

Kill empty statement.

Make BacktraceImpl::resolve a bit faster.

Print the full frame information all at once.

Replace void* used for RemoteProcess pointers.

Use a RemotePtr<> for RemoteProcess::heap_address.

Stop using void* for xbt_mheap_t.

Fix cloud.xml: symmetrical routes are implicit.

Add file to basic_parsing_test so that it doesn't remain untested..

Make NetworkConstantAction "final": virtual set_state() called from constructor.

Make virtual functions called from constructor "final".

Sonar S1699: Constructors and destructors should only use defined methods and fields

please sonar

Display leak origin in message, if -trace-call-location is used.
+ downgrade messages to INFO.

hide references to current_handle inside *.cpp file, to avoid breaking if SMPI is not compiled

appease 32 bits

Use a Remote<> for remote smpi::Request.

appease clang

change the way we display leak types, skipping demangling

try to improve handling of comm_self a bit.

fix for potentially empty names

Add the option to print origin of a failing handle.
This uses trace-call-location flag for smpicc, store the value at handle creation, and tries to return the most relevant one on a crash (that's actually a pain and will have to be checked)

add a name() to all F2C objects.
This avoids demangling need for leak check (and should fix tests, as freebsd has a broken demangler on our ci system).

oops

same for win, get_info needs a copy.

remove unwanted s4u include in all smpi files

get_info calls actually must return a copy.

generalize mark_as_deleted call

Don't add handles to the lookup for the sake of just removing them immediately...
if they're not there .. well they are not.

add two fortran declarations

Add message to help improve backtrace

\t removed

print backtrace when we exit due to an MPI failure

factor on_creation signal trigger in Host::seal()

prefer create_host than creating the creation args

cleanups

simplify kernel part of disk. It's really shallow now

cosmetics

Match Io and Exec creation

Save one assert and do not use shring policy to determine if WifiLink

cosmetics

Single way to create links in NetZone

With the refactoring, we don't need 2 methods in NetZoneImpl to create
new links.
The link type (wifi or wired) is determined by the NetZone type
(WifiZone).

Simplify NetZone's hierarchy

A NetZone is RoutingMode::base by default. When we add a child to it, it
becomes RoutingMode::recursive.

One method to set netzone's relationship

A netzone is child of its parent, set_parent() is enough to determine
the relationship parent/child of netzones.

Copy models from parent netzone

Allows user creating new netzones without explicitly setting models

Minor fix in doc

[appveyor] Stick to the stable branch of pybind11.

Missing space (?).

checks for PMPI_Type_create_subarray

check for alloc mem

check illegal in_place for gather/scatter
also stop calling pid rank, it's confusing.

check ranks for probes/iprobes

increase rank checking for RMA comms

this test is invalid with 1 process. Align with mpich and use 8.

group inputs validation

set keyval to invalid after free

MPI_Abort : try to kill other processes from the communicator.
No idea if this helps

Keyvals : handle MPI_KEYVAL_INVALID

Errhandler : inputs + cleanup

oops.

Abort : check input, and remove fixme... Actual implementations do the same

Init : dont just crash with asserts, return errors (they will crash if errhandler is configured to)

comm, input validation

use MPI_COMM_WORLD errhandler if MPI_COMM_NULL is given

Fix null dereference.

Sort the list of leaks before print.

avoid allocating comm_self if not needed

Tell when we have more leaked handles to display