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

cosmetics

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.