Release v3.28

Fix make distcheck

doc: Fix a subtle formatting error that is not reported by tools

Small improvements to the changelog and doc

I think the changelog is now complete

proposal for a NEWS file for the upcoming release

try to further improve docs/find-missing

Remove useless includes from the public headers

doc: praise for 'discovery reports' as a way to contribute back to the project

also, reword and shorten some other sections in the same file

remove 2 files with no useful content to scavenge from the old doc

Hide some deprecated symbols from doxygen

port find-missing.ignore from autodoxy to breathe

Use == for string comparison [ci-skip].

Simplify, and avoid nested SmpiBenchGuard when rastro_resolution is called by rastro_timestamp.

[sonar] Less implicit conversions.

Add network factors in changelog

doc: fix an issue spotted by find-missing.py

docs: properly report typedefs that are missing in the doc

Adapt docs/find-missing to breathe now that autodoxy is gone

Fix another bunch of warnings in doc generation.

Use 'console' syntax highlighting for shell typescripts.

Various corrections in docs.

Missing const.

Fix debug message better [ci-skip]

Remove dead entries in 'docs/find-missing.ignore'.
Kill unused file 'ignored_symbols'.

Instr: for consistency, rename 'father' to 'parent' here too.

NetZone: rename the remaining occurrence of 'father' to 'parent'.

Cosmetics around deprecated functions.

Kill deprecated private field ActorImpl::userdata_ (use Actor::get/set_data instead).

This is on the *Impl side, and does not need to wait for the deprecation quarantine.

Make comment match the declaration in Engine.hpp.

Vector::at() is good enough to check bounds.

Reduce number of unnecessary copies of 'args'.

Make EngineImpl::tasksTemp a local variable.

Simplify function calls (avoid explicit std::vector<double>).

Either use the variant taking a scalar, or use an initialization list.

Make single argument constructor LinkInRoute::LinkInRoute(const Link*) explicit.

Fix empty #ifndef..#endif. Annotate helper function as deprecated.

SplitDuplexLinkImpl always have SharingPolicy::SPLITDUPLEX. Use a sdingle assert in Link::get_impl().

doc: autodoxy is gone, so remove a workaround for it

Cosmetics in ChangeLog (mostly reorder and rewrap)

small doc improvement

Use reference to const for large input parameters.

Add 'const' for sonar.

Add informative comment near implementation for deprecated functions.

Attribute 'virtual' is redundant with 'override'.

It's an internal error if get_impl returns null. Add an assert to be sure.

(and hope to please fbinfer)

Changelog: C++ interface

Sonar variety

Try to fix opensuse, gcc 11.0.0 build

Merge branch 'split_link_impl' into 'master'

Split-Duplex: s4u support

See merge request simgrid/simgrid!66

Split-Duplex: new management

1. Huge commit to add support to Split-Duplex in S4U.
  - New interface: s4u::SplitDuplexLink

2. Allows its creation through the C++ interface without manually creating
the link UP and DOWN.
  - NetZone::create_split_duplex_link

3. Changes in NetZone::add_route to allow the description of the
direction in C++ interface
  - LinkInRoute: wrap around Link* to indicate the direction
(UP/DOWN/NONE) for the link.
  - UP/DOWN are used to split-duplex links
  - NONE for other types (shared, wifi, fat-pipe)

4. Fix use of split-duplex links in symmetric routes in XML too.
     <route src="alice" dst="bob" symmetrical="YES">
       <link_ctn id="link1" direction="UP"/>
     </route>
  Now this code works, it will add the link1_DOWN as route to
bob->alice. In the past, it would add link1_UP for bob->alice too.

Don't mix LMM sharing policy with S4U.

Add another enum for maxmin sharing policy
In the maxmin system, only FATPIPE and SHARED make sense.

Don't mix C with C++ API here.

This is in the Tutorial section and should be kept simple.

Fix another bunch of dead links in docs.

More trivial fixes for docs.

Remove reference to Mutex/Semaphore ctor and dtor, since they are not public.

Increase requirements for breathe.

Disallow sphinx>=4.0 for now: it's breaking on CI

Exception occurred:
  File /usr/lib/python3/dist-packages/docutils/nodes.py, line 625, in __getitem__
    return self.attributes[key]
KeyError: 'domain'

Try to increase sphinx requirements.

Match version currently available in Debian testing and unstable.

Minor fixes in doc.

Two more const methods spotted by sonar.

More int -> unsigned long.

Don't fail when there's no error!

Fix a few warnings when building doc (mostly related to doxygen).

Correctly remove generated files.

The added "rm" command for s4u-exec-ptasks prevents use of ADD_TESH_FACTORIES
(it chokes on "--cfg=contexts/factory:...").

Get back |br| in XML doc

Fix my previous mistake removing them

This is an unsigned integer.

GraphNodeData::graph_id_ is an unsigned long.

Type consistency.

This is a test for emptiness.

[sonar] Avoid C-style array.

Parameter 'fd' is always -1 for xbt_mheap_new. Kill dead code.

Several improvements in doc [ci-skip]

Declare method const.

Fix type for variable.

Use std::unique_ptr and remove explicit destructor.

Pass the real pointer to the englobing MC class for the event callback.

(avoid accidental upcast)

Use ssize_t for Exec::wait_any and Io::wait_any too.

Use ssize_t as return type for Comm::wait_any.

Use ssize_t as return type for Comm::test_any.

Update .mailmap.

Two more s/pointer/reference/ for std::vector parameter.

Prefer a reference for first parameter of {test,wait}_{all,any}.

Add a test in teshsuite for Comm::wait_all_for.

Implement Comm::wait_all_for (wait_all with a timeout).

Reduce code duplication.

Make field 'mc::ActorState::times_considered' private.

Make field 's4u::Host::pimpl_cpu' private.

Add missing copyright notices.

help latest fedora with mc

be a bit more safe here

changelog update for SMPI

document _TAG versions of sampling macros

Fix distcheck

Link::set_concurrency_limit

Allow setting a limit for concurrent flows on platforms links.

Add a simple example limiting the number of flows to 2 in the single
platform link.

Small entry about dynamic network factors [doc]

StarZone documentation [doc]

Explain Star Zone and its special route declaration.

Fix some links in graphical toc [doc]

Adding some packages missing packages [doc]

Packages needed to run ./Build.sh script

Adding C++ platform documentation [doc]

- Point to the examples.
- Rewrite routing part to explain how we calculate the routes between
different zones.

Add a SMPI_SAMPLE_LOCAL_TAG and SMPI_SAMPLE_GLOBAL_TAG macro for sampling, to provide unique parameters to separate various calls to sampling.
This can be used when a kernel is called with various distinct sets of parameters.
Tag is a string of max size 128.

Try to be more coherent, fix misleading message, and fix test.
It was asking 0 iterations max, let's say that this means "exit after the first".

fix bug in sampling, it was ignoring max number of iterations