Reduce scope for variable.

Global--.

Another bunch of int -> size_t conversion.

aid_t for src and dst in smpi::Request.

Convert some int -> size_t.

Prefer emplace to insert.

For Sonar.

More aid_t.

Avoid alternative keywords for boolean expressions.

Plus slight simplifications.

Reference for std::vector parameters.

Fix bound checking.

Combine identical conditions.

Use std algorithm.

Useless indirection.

Sonar const's...

Convert int -> size_t for TRACE_smpi_send (and below).

More int -> aid_t.

Use type aid_t for process id. in instr_smpi.

Adapt leak example to new display.
Add another leak and a size change between processes to see what it shows.

Group buffer leaks by origin as well.
Keep total size and min/max for each if possible, then display by largest total size.

Plug memory leak.

Const, once again.

Doc improvement [ci-skip]

Merge branch 'master' of framagit.org:simgrid/simgrid

Finally rename smpi::Group::actor_pid() back to actor().

Kill smpi::Group::rank(s4u::Actor*).

Kill smpi::Group::actor(int).

Change smpi::Group::set_mapping to take a pid.

Add assert's to ensure consistency between the two kinds of mapping.

Define new rank_to_pid_map_.

Lookup for parent process (like in Group::rank(s4u::Actor*).

In smpi::Group, 'index' is in fact a PID. Fix type and rename.

Kill a TODO (-1 is correct for an invalid PID).

MPI_SOURCE is a rank; src_traced should be a PID.

Fix comment.

Pass large parameter by const reference.

Minor cleanups (useless stuff).

also requalify mc version of the tesh

Group leak reports by type/file/line (for handles, will see for buffer later).
And split reporting.

Sonar polishing.

Pointer-to-const parameters in smpi::Group.

Reduce code duplication (and fix bug in Group::difference).

A for-loop with 'num' modified in the body is too complex according to Sonar.

Fix smpi::Group::intersection().

Using 'size2' for controlling the first loop was a bug (spotted by Sonar).

Fix Group::group_union().

Included ranks from group2 were not good.

Simplify smpi::Group::difference().

Review smpi::Group::*excl() and smpi::Group::*incl().

Mostly cosmetic changes.

Kill redundant member 'size_' from smpi::Group.

Minor changes in smpi::Group.

Remove unused option from example.

Simplify loop searching for a bypass route.

Prefer a do-while in this case, and rewrite without 'break'.

Use std::find_if when looking for a matching comm.

Replace C-style arrays and remove const_cast.

Define local class EngineWrapper to encapsulate common code.

Use const& for std::string parameters.

Pointer-to-const.

Fix build with older clang.

please clang take 2

please clang

Some code improvements.

Avoid unnecessary map lookups.

Thanks @agiersch for the review

add wrappers with scalars on the user side

simplification

cosmetics

cosmetics

s4u::NetZone::create_router.

Export create_router to s4u so we can use in C++ platforms.

Please sonar, just a little

Use rule-of-three, and hope to please sonar without breaking clang13 builds.

Revert "Delete the implicitely defined move constructor."

This reverts commit c26d236dedf486357690800a0d5527871ef45620.

Clang 13 fails with "The specified type does not meet the requirements of
Cpp17MoveInsertable"

AFAICT, the std::deque makes a noexcept constructor impossible.

Finally provide a noexcept move constructor.

Previous commit (f7cf88e9b8 Delete the implicitely defined move constructor.)
was causing clang 13 to fail with: "The specified type does not meet the
requirements of Cpp17MoveInsertable".

Use emplace_back to add a new property_set.

Sonar S6003.

Minor Sonar smells (const, etc).

Move a \n.

Delete the implicitely defined move constructor.

Fix for sonar issue S5018: Move and swap operations should be "noexcept"

Disable copy for actor::ProcessArg.

Also fix sonar issue S5018: Move and swap operations should be "noexcept"

Delete the implicitely defined move constructor.

Fix for sonar issue S5018: Move and swap operations should be "noexcept"

Anyway, AFAICT, the implicit move constructor here does nothing more than the copy-constructor.

Sonar requires a comment for empty functions.

Simplify add_route methods.

Leave only one add_route method, as it was initially.
Deprecate old add_route which uses LinkImpl*, s4u is the interface we want
with user.

Vivaldi on top of StarZone

Implement Vivaldi on top of StarZone.
This allows the easy usage of Vivaldi on C++ platforms (no need to
access specific methods from Vivaldi NetZone.

Platform creation from XML is unchanged.

Adjust add_route to accept route between netzones

StarZone: implement get_graph method

Add get_graph for StarZone. A top node named with the zone's name is
created to be the center of the star.

New netzone: Star Zone

Implements a netzone following a star topology.
Components inside the Star topology are connected by default (through no
links), users can add routes using the NetZone::add_route method:
- netpoint -> nullptr(*): links used in all outgoing communications
- nullptr(*) -> netpoint: links used in all incoming communication
- netpoint -> netpoint: loopback

Duplicated links in a route inside the Star Zone are removed.

Added UTs for StarZone, some tests are disabled since they check the
xbt_assert and would stop the test execution.

[ci-skip] Update lua tesh files with deprecation warning.

constify

this is no useless

do not call sg_platf_new_link when creating Cluster-ed Zones

improve router creation

Deprecate Lua platform files.

No need to duplicate variables.

Const...

change way disks are managed in the XML parsing

Define CommImpl::wait_for().

Print state_ in symbolic form.

More cosmetics around namespaces.

Fix segfault with lua.

Inline function.

Define CommImp::wait_any_for().

Cosmetics.

please clang (?) and make properties optional to create an ActorImpl

do not allocate/free properties

do not allocate/free radicals