Try to help sonar.

Fix mc liveness examples.

Objectify MSG task send

Simplifications in MSG task execution

please sonar

remove or deprecate now useless code

please clang

s4u::Exec now has 2 child classes
s4u::ExecSeq and s4u::ExecPar

side effects:
+ no more C-style host list in HostImpl and ptaskLO7
+ Sequential execution can have a timeout
TODO:
+ get rid of ExecImplPtr in MSG
+ check and clean

Parmap status is simply a boolean.

Use std::function instead of function pointer.

Use proper type instead of void*.

Assert that the pointer is not null when dereferenced.

Field is initialized by constructor.

those pimpl are actually never null, stop fooling sonar

apply agier fixes for StarPU build

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

don't let sonar think that pimpl_ can be null

sonar: pass large parameters as const; don't name unused params

Sanitize the prototype of Actor::on_exit() callbacks

Changes on_exit(function<void(int failed, void*data)>, data)
into    on_exit(function<void(bool failed)>)

So, we are using bool for boolean values, and we don't force the user
to build a closure on void* data manually. If you want to pass a data
to the callback, just use the C++ formalism: [data](bool failed){...}

Other changes on the way (that excalated quickly)
- Document some recent changes to the internals in ChangeLog
- Document this_actor::on_exit() and Actor::on_exit() in doxygen
- Also sanitize the prototype of MSG_process_on_exit(). Was taking a
  (void*,void*) function; now takes a (int, void*) one. This was too
  weird to use void* to denote integers -- I actually doubt it ever
  worked the way it was written, but the tests are too poor to be
  sure: the only usage of this function ignores the return value.
- Also add a new test in activity-lifetime: test_sleep_restart_end
  Commented for now as it breaks miserably.

Move ForcefulKillException to the root namespace, along with the other exceptions

Rename StopRequest into ForcefulKillException to contribute to #325

further convert some bits of the documentation

snake case this bool

cosmetics

more factoring when sending MSG tasks

Add a comment to explain previous changes.

Fix memleak.

Fix use of invalidated iterator.

Use C++11 <random> for simcall mc_random.

Avoid shadowing outer variables.

Kill unused parameter and please Sonar.

Use specialized vector::swap.

Use Rvalue references for exception constructors.

It seems to be sufficient for now.
Maybe one day we'll need to complete to handle Lvalues.

Stop setting random seed at initialization.

kill obsolete doc

There is no need to convert that.

convert another page from old doc to RST

doc improvement

I hope that this will clarify #325, and #132.

mv the internal isend function to the class

remove useless simcall for IO too and use better casts

one step further not using kernel execution activities in MSG.

The creation of the pimpl needs no simcall

Now that init and start are split, creating the ExecImpl in init (the
constructor) has no impact on the simulation kernel. The action is
created by start().

This slightly modifies the ordering of the output in one tesh file

Yell when there is an exception, not when there isn't

mess with MSG tasks

Make destructors useless.

Make capture list explicit.

Empty callback is optional.

Hide std::free to Sonar.

Ignore Sonar issue "Macros should not be used to define constants" in *.h.

Use default initialization.

No static analyzers, myCoords[] and targetCoords[] don't contain garbage.

Save dimensions_.size() in a local variable to help Sonar and Clang, plus other
simplifications.

Don't rely on random to generate temporary file names.

Use C++11 <random>  instead of rand().

Remove never used FindRngStream.cmake.

better message for this exception

add test related to #220 + fix killsend

If the test and the launched actors share a scheduling round, the
outer test comes first and execute the asserts before the actors are
able to end their execution. As we want to control the status of the
actors when they end, let the outer test wait a bit before asserting.

change exception raised if a StopRequest is catched in Host.off()

activity-lifecycle: test exec just as we test sleep

We should also try to cancel all these nice actions. And at some
point, we should probably come up with a DSL or find another way to
factorize some code.

Don't give actors a chance to survive their exec if their host is turned off

Before, they received an HostFailureException that they could catch to
survive. Now, they are simply killed (once their on_exit callbacks are
executed).

This is more consistent with what happens with sleep actions, and thus
fixes the github issue #325.

please sonar: don't declare an empty callback

Fix forwarding of the program arguments.

Use a static storage to avoid dangling pointers, and ensure that there is no
out of bound array access.

Fix Backtrace::operator=.

Anonymize unused parameters.

Enforce "Rule-of-Three/Five".

Avoid null pointer dereference.

surf_parse_error() never returns, remove dead code after that.

Remove dead code after exception throw.

Remove dead code after returrn statement.

Method is redundant with base class.

Mark methods as "override".

Capitalize constants.

Replace #define with constexpr declarations.

Kill unused constants.

fix (#220) ?

If an actor ends right after a dsend, the Comm should not be canceled
to let the receiver a chance to catch it.

It was removed in Comm::cancel(), now it's not, and the destructor has
to be a bit more careful.

As the comm was canceled, if the receive is posted later, it nevers
find its matching send and may stall forever or timeout.

This changed the output of the chord example (using dsend), but I
suspect it was not so correct before with some dsend being canceled
and the corresponding receive ending on timeout.

This also highlighted an issue in Java. The cleanup function for dsend
is now called at the end of the simulation and get_current_thread_env
crashed (because the current thread is maestro).

Hopefully, this commit does not cause new leaks.

make MSG_process_create die if host is off.
This was requested by adsein long time ago.

fix typos in warning

Change color of "barrier" in trace. It was too close to "computing"

Add simple gemm example for SMPI with sampling macros.
Adapted from the proxy app from Codevault.
This could be a good basis for the tuto

Yet again change the way SMPI_SAMPLE_MACRO works
There is no need anymore to have the body of the loop inside the macro. Kudos to Arnaud L for the ideas.

now usage is just turning
for(x;y;z) into SMPI_SAMPLE_GLOBAL(x,y,z,iters,thres)

Time injection will be done only once for all the section, speeding up the simulation.

TODO : iterator value will not be correct at the end of the loop for now.
Not important if iterator is local to the loop, but may be annoying if not.

fix (#324)
Throw a HostFailureException when one tries to create (or attach) an
actor an a host that is not on.
If this happens during the parsing of the deployment file, just let it
go.
for MSG, we could die if this happens (we don't for now)

Build still broken with ld 2.31.1.  Check only version for gcc.

[sonar] Improve Cognitive Complexity.

NO_MAX_DURATION is a special value, use (in)equality for testing.

Move common test to Action::update_max_duration.

Kill now unused functions SIMIX_mutex_{un}ref.

Convert last call to SIMIX_mutex_unref.

fix last issues
+ counter_ is used by heaps, use id_ instead
+ fix dumb find/replace

end objectification of MSG tasks

mv counter from msg_task_t to simgrid::msg::Task
this was the last field, MSG tasks are now objects
have to remove the useless layer now!

mv data from msg_task_t to simgrid::msg::Task

simplify these tests

mv category from msg_task_t to simgrid::msg::Task

move name from msg_task_t to simgrid::msg::Task

do not promote bad practices: use "accessor" to get task name

objectify simdata_task_t

priority and bound of MSG tasks can only be set BEFORE execution

this is the rule for S4U, should be the same for MSG.
BTW, the behavior of the LMM is likely to be confused by such dynamic
changes during execution.

Side effect: we should remove the setters in ExecImpl, but there are
used by two simcalls that are tagged as deprected with v3.25

source is sender's host. One field is enough

play with (parallel) execs. still not satisfying

SIMIX_mutex_{un}ref become MutexImpl::{un}ref

dont call HANDLER

please sonar