Misc issues in shell scripts (codefator.io).

Fix potential memory leaks.

wifi: make room for the ns3-wifi binding

smpi topo : a topo can be shared between some comm (duplicates), but there was no refcounting.
Try to switch to a shared_ptr to avoid segfaults or leaks...

that's not a double

fortran : protect in case some calls return success, but with a null request.
This can be caused by sending messages to unreachable processes.

attempt to please fortran on ci

Trigger a warning when abort is called and a process exits.
Does not hurt and can help understand why there is a deadlock in simgrid, when the app forgets to print an error message.

dist

MPI IO : "support" for ./ in filename

MPI IO: now with more fortran bindings

fortran support for MPI_UB and MPI_LB

fortran : allow use of C names for types

Remove redundant blank lines (codefactor).

Make smpif90 support SMPI_NO_UNDEFINED_CHECK too.

SMPI : Allow to deactivate link-time definitions check with an environment var
This is breaking compilation when the code has shared libraries itself (CLAMR).
Not sure if we should keep all this, actually ...

Don't use XBT_LOG macros within a function (thanks lgtm for noticing)

lgtm: Add some missing header guards

Better explanations for the parameters of the decay model

MC: inline a function

bump osx version on travis

fix ns3 builds

remove an extraneous ';' that fools codacy

Missing const marker

Save the src_wifi_link and dst_wifi_link in a subclass of NetCm02Action

This way, interested parties won't have to recompute that information
and possibly miscompute it.

fix how wifi's host_rate is used in communicate()

- Enforce that we use the first provided link for the src host,
  and the last one for the dst host
- Assert that the provided links are actually associated to by the right host
- Fix the test that was missing an association

fix save src/dst

remove the src and dst from the Link::on_communicate callback since it's now in the action

save the source and destination hosts in the NetworkAction

stick to our coding standards: getters should have get_ in their name

third time's the charm. It was already done properly for smpicc, so just port it to smpicxx

disable on osx as -z is not supported there

add definitions check to link phase in SMPI (gh #342).. let's see how many ci workers complain

sonar: also ignore S5271 for the examples in C++

Plus fix some other warnings reported by sonar.

Please the checkers (sonar, codacy)

- reduce the scope of a variable
- Mark a constructor as explicit
- Blindly obey the given hints about javascript that I don't really master

do not downcast activities to the wrong type just to get their name

- I use a protected method to keep my data private. Doing otherwise
  would trigger sonar to complain.
- The code is not particularly clear this way, but I fail to see how
  to do otherwise: I don't want the getter to become a virtual method.
- This was reported by UndefSan now that we have a test that deadlocks.

Better fix for FG#50

- The previous solution was not allowing the suspended actor to be resumed properly
- Instead of going for a complex story where we create an activity on
  which to suspend the actor, simply unschedule it on suspend(), and
  explicitely reschedule it on resume()
- This is much easier since we don't have to save and restore the
  simcall of the suspended actor, even if the price to pay is the
  explicit simix_global->actors_to_run.push_back(this); in resume()
  of which I'm not a big fan.

Fixes https://framagit.org/simgrid/simgrid/-/issues/50

please codacy by not setting a variable we won't use

sonar: ignore more useless warnings in the examples

Fix MC on FreeBSD

Fix pip builds

stick to our coding standards: fields must have a trailing _

sonar: ignore more crufty warnings about the examples

The examples must be readable. I'll shut off sonar on these issues

MC: obey to the rule of 5 to please sonar

Fix MC+SMPI builds

Ensure that suspending an actor running later in the current scheduling round actually works

Fixes https://framagit.org/simgrid/simgrid/-/issues/50

cosmetics

MC: Further renamings

try to print pointer correctly on sunos as well

doc: add units for message sizes, and try to make smpi/buffering more understandable

MC: actually remove the comm channel from RemoteClientMemory

It is now in remote::CheckerSide. This makes it symmetric with remote::ClientSide

There is still a lot to do to clean things up:
- the actual communication Checker->App is spread all over the place
- the ModelChecker singleton dupplicates the function of the mc::Session
- plus the other ugly things that I fail to see right now given the ambiant mess

MC: rename remote/RemoteClient -> remote/RemoteClientMemory and remove networking from its responsability

MC: move a function around + cosmetics

MC: inline a trivial function

MC: rename remote/EventLoop into remote/CheckerSide

MC: rename remote/Client into remote/AppSide

The return type of the sg_actor_on_exit() callbacks is now 'void'

actor-exiting: Align the C test with the C++ one

the on_exit() of each actor is also executed when the simulation deadlocks

Extra header needed by FreeBSD to know SIGCHILD

split a piece of src/mc/ModelChecker.cpp into src/mc/remote/EventLoop.cpp

MC: tiny code simplifications

further simplify the MC initialization

simplify the MC initialization by using only one env variable

Remove a useless call in MC initialization

That's a very long time that we don't have two heap regions anymore.

Fix doc markup to not break the build process

make sure that the doc build failures get detected

Use const reference for std::string parameters.

Also check that the i/o operation didn't fail.

Update ChangeLog.

Merge branch 'random_readwritestate' into 'master'

[xbt/random] Read/Write the state of the RNG

See merge request simgrid/simgrid!36

revalidate tesh files

use signals for smpi tracing of execs

refactoring SMPI trace initialization

Tests whether the file could be opened and removes the temporary file at the end

Now returns true if the file could be opened

Tries to handle IO errors

remove redundant Link Type

namespacing cosmetics

more lambdas

Kill obsolete option --cfg=contexts/parallel-threshold.

The associated code was removed 4 years ago by commit
9ed461c11fd10eeea596831f2791e359c05d26bf "kill more old cruft: adaptative threshold".

Update .mailmap.

Merged FG!35.

Merge branch 'doc' into 'master'

Add a modeling hint for parallel links in doc

See merge request simgrid/simgrid!35

Merged GH!339.

Added Mailbox::set_receiver to python binding

Add a modeling hint for parallel links in doc

Explain how to model several parallel links between two routers
(usually useful when modeling fat-tree platforms for example)

add the capacicity to modify links' latency and bandwidth

please sonar

Typo fix

Unit test to check if the RNG state is written and read correctly

It is now possible to read/write the RNG state in a file.

use signals on PajeEvent creation/destruction

still fighting against this code

useless morning cleanups

fix that example

set_maestro example: improve comments

cmake: code simplification (hope it does not break)

fix distcheck