Reduce scope for variable 'rank'.

Cosmetic rewrite to please cppcheck.

Fix: Condition 'slow_copy' is always true.

Reset pointer to make it clear for cppcheck that it's not freed twice.

Initialized all struct members.

Currently not tested, but should fix smpi_shared_free().

Cosmetic rewrite to please cppcheck.

Fix: Variable 'rem_size' is assigned a value that is never used.

[SMPI] Fix iprobe bound

While looping over iprobes, not the whole CPU can be used.

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

[codacy/cppcheck] A few more issues.

[codacy/cppcheck] A few more issues.

Merge remote-tracking branch 'origin/master'

fix clang builds

Merge remote-tracking branch 'origin/master'

[codacy/cppcheck] Potential memory leak.

think about users

[codacy/cppcheck] Fix: Variable is reassigned a value before the old one has been used.

[codacy/cppcheck] Reduce scope for variables.

[codacy] minor issues in shell scripts.

codacy.yml: complete and sort exclusion list.

Alsot exclude python scripts for sphinx in docs/ subdirectory.

[codacy] Add quotes around exclude paths in .codacy.yml.

set_kill_time is a method of ActorImpl

Engine is in charge of platform creation, not SIMIX anymore

move code to the right place

further cleaning in simcalls

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

modern simcall for set_category

allow all activities to set their tracing category

modernize a few simcall_execution_* calls

sanitize the OOP of kernel::profile

* Add a Profile::next() returning the next event of the list.
  This opens the way to future profile kinds, based on statistical
  generator instead of explicit lists.
  Earlier, FES was advancing the internal pointer of the Profile.

* Change FutureEvtSet::add_trace(Profile, Resource) into Profile::schedule(FutureEvtSet, Resource).
  This way, Profile knows about its FES and there is no need for a
  global (also making testing easier), and FES does not have to mess
  with the internals of Profile.

Fix (old) clang builds

Fix lua build

sonar: simplify this code

sonar: Replace this if-then-else statement by a single statement

profile: cosmetics: inline a function and rename a variable

s:k:profile: turn two functions into static methods

align file position to the C++ namespaces

properly split init/start for Exec activities

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

make sure to define _FILE_OFFSET_BITS as early as possible

It won't be effective if defined after the system headers are loaded.

Please codacy, forget about Catch2

doc: reword to remove crufts of the old cunit

tidy some #includes in MC

Please Sonar.

Remove last references to testall.

factor surf_action_ in ActivityImpl

uniform naming across activities

cosmetics

[sonar] Split declarations, unshadow variables.

RIP cunit.

Convert the unit tests of xbt::config to Catch2.

Convert the unit tests of xbt::str to Catch2.

Convert the unit tests of xbt::dict to Catch2.

Reduce number of #include.

Check for emptyness, and populate dynar before testing xbt_dynar_foreach_ptr.

properly split init and start for IO activities

Ignore Catch2 in sonar

Revert "Try to improve coverage reports for MC tests."

This reverts commit db3df03114cb65a9c27c2c1dfd6faf964b571254.

The following tests are broken (timeout): mc-bugged1-liveness-visited-ucontext,
mc-bugged1-liveness-visited-ucontext-sparse and mc-umpire-comm-dup-no-free2.

To reproduce, it seems that compile_optimizations should be disabled.

fix makedist

Convert the unit tests of xbt::dynar to Catch2

cosmetics

Try to improve coverage reports for MC tests.

Define coverage_checkpoint().

It's defined as an alias for __gcov_flush() when coverage is enabled, or a
no-op otherwise.

Define COVERAGE for C++ too.

Remove unused static functions.

Refactorize and speedup a unit test

I don't think there is a real need to test each region size from 1 to
256. Testing on [1;32[ should give a sufficient coverage, I think.

Remove the now useless unit_test_framework

convert the two other boost::unit_test suites to Catch2

Convert a first unit test to Catch

Add the Catch header

This is a header-only library for unit-testing. Including the header
in our project is the recommended way of using it. No need to test for
its presence, then.

Boost::unit_test_framework shall soon vanish from our tree. Testing
for its presence is too bothersome, and Catch seems easier to use
anyway.

sonar: kill unused imports in java

uncomment forgotten line, includes smpi_helper in C++ codes as well

avoid sending misleading log message

Our cmake files are so fragile, fix last changes.

[jenkins] Temporary disable LTO on NixOS/gcc 7.4.0/ld 2.30.

Optimize MC builds a bit further.

ContextUnix belongs to Simix.

Include cmake/Flags.cmake after DefinePackages.

The former uses variables defined in the latter (e.g. MC_SRC).

Improve coverage, even with high log threshold.

Kill dead code in kademlia examples.

[java] Delete broken test.

[java] Catch StopRequest during VM migration.

[sonar] Inform that some properties are overridden by Jenkins.

[sonar] Only exclude from coverage a subset of python files.

Enable running parmap_bench for coverage.

parmap_bench: make timeout a runtime parameter.

Kill dead code.

This test for mallocators has never been used.

Process.java: reduce the debugging scaffolding now that it works

this file should not be executable

Only unregister the kill timer once, in maestro context

Remove workaround for g++ 4.8.

g++ 5.0 or higher is advertised in the doc.

Implement xbt::Result with  boost::variant.

std::variant is today not an option because it's C++17.

Fix memleak.

The comment seems to be invalidated by commit 4f2d08b0ae4625d32890dc4613e1d842ad0f7b77
"Sanitize the API of this_actor::parallel_execute()"

Document a recent example

activity::CommImpl: stick to our naming standards for the fields

help scan-build understanding that status is not used unintialized

Properly call Actor::on_destruction() in maestro context

This induces a harmless change in trace-process-migration.tesh and
fixes the s4u-actor-exiting example \o/

inline a function

no need to remove the killed actor from canceled comm: refcounting is working

refcounting on the src and dst actors of a comm

The pimpl of s4u objects is const

But the one of activities and the cpu_pimpl of host (the codebase is
not ready yet).

always cancel the comms of killed actors in the context of that actor (not in maestro)