woops

merry chrismas

cosmetics

clang-format these files, no change

remove a few smells in instrumentation

please sonar: name parameters

fix clang builds by including <string> where needed

plug a huge memleak

I forgot once again that xbt_dict actually delete their content when
you empty it, but not the std:: containers.

finish the cleanup of the netcard list container

- move it from global to the engine
- use a std::unordered_map instead of a xbt_dict. Its order being
  harder to predict, this forces to sort the list in the routing
  integration tests.

I'm not completely happy about the register/unregister functions being
public, but I cannot find anything better, and it's later already...

have only one callback for link creation

inline another function, plus some cleanups

try to fix clang++ builds

inline a useless function

further trim the includes

Check that scope is non-NULL pointer.

change all links to my old webpage

please sonar

another bunch of include cuts. The last one for today

Fix windows build

netcards_dict is not visible enough for windows anymore, so don't use it.

further reduce the amount of includes

cut some useless includes

drop unimplementd VM methods: save/restore

cosmetics

VM: don't take ignored parameter that may fool the users

convert the last vm simcall into s4u code

VM: rename a symbol and improve comments

convert vm_suspend from SMX to S4U

convert another VM simcall into a s4u method

VM: move content from simix to s4u

cosmetics

explicitely capture variables in lambda to please sonar

cosmetics

Two variables must be defined without using cmakedefine macros.

They have values other than 0|1, mea culpa...

disable mc for sanitizers to have a cleaner output

Use the CMake macro #cmakedefine01 in configurable files.

cmakedefine01 allows to perform C preprocessor replacements
that define a variable to 0 or 1 when configure_file is called,
instead of (not) defining the variable (see
https://cmake.org/cmake/help/v2.8.8/cmake.html#command:configure_file
for the 2.8.8 doc).

This ease the defining of config.h variables (no foreach loops
that preset them to 0), but two requirements must be met:
 1. variable name must match in cmake and config.h.in
 2. no comment on the macro line

Prepare the next release

move content from MSG_VM to s4u::VM

reduce the amount of static casts

improve test (and stop using unqualified printf)

cosmetics

- less new/delete when avoidable
- rename some misnamed variables

new function: Engine::netcardList()

Move all netcards into the dict, and the dict to the engine

use std::string instead of bprintf

reduce the public exposure of netcards_dict

no need for a lib to store the netcards. A dict is easier

remove surf_routing.hpp

next release is 3.14

please sonar on yesterday's commits

you mean I should compile the code before commiting? Hu ho, ok...

Completely rework the properties of netzones

- This changes some MSG functions, but I doubt that any users really
  used them. Even if someone did, there is no real loss of
  functionnality. The only missing thing would be if someone wants to
  traverse the whole set, but I don't have no good idea here since the
  data is stored in a C++ container. It's hard to let C code traverse
  it. I could add a _dump() method to just display it.
- the change in the tesh file denotes an old bug that this change
  fixes. The name property was not defined in the cluster tag, but in
  the previous one in the XML, so it should not have landed in the
  properties of the cluster's hosts.
- the code in the XML to stack the dictionnaries for each AS is simply
  gone. The AS is created in the STag function, so simply retrieve it
  by name and use it when we need it. No need for an additional storage.
- surf_routing is almost gone, now \o/

bypass MSG in the java bindings

msg_as_t  ->  msg_netzone_t

small cosmetics advised by sonar

MC: ignore a new boost-related library

The new lib is libboost_context-mt, but clang-format insisted on
making my patch unreadable.

fix MC build with paranoid flags

fix jedule builds

start rewriting the platform documentation now that AS are called netzones

start reformating the manual for the AS -> NetZone transition (ignorable changes)

update function documentation and variable names in the As -> NetZone transition

rename the class objects and C types for the As -> NetZone transition

clang-format changes related to the previous commit

start renaming AS to NetZone

These things are obviously not Autonomous Systems as in Internet:
they form a hierarchy...

That's an intrusive change ongoing, but we want the best for SimGrid,
don't we?

continue emptying the surf_routing files

move some storage-related content out of routing into storage area

sonar: try harder to not choke on casting pointers into literals

cosmetics: .name().c_str() becomes .cname()

sonar: ignore 'Pointers should not be cast to integral types' and bump version to 3.13.91

don't fool sonar if we can avoid it

these are redundent tests of which we already have the answer

don't test pointers after dereferencing them (thanks, sonar)

try to not disable our logs on sonar, just to see

revalidate this test now that I updated the corresponding errmsg

please sonar: dont apply binary operators to signed ints

improve an error message

typo

please sonar, and other small cleanups

- Init members during initialization
- One operation per line at most
- Don't let fields uninited after the constructor
- Obey our naming schema (fields' name end with _)

kill the s_surf_parsing_link_up_down_t datatype

cosmetics

use std::pair instead of s_surf_parsing_link_up_down_t for Cluster private links

revalidate the surf_usage tests

These tests are meaningless anyway and should be replaced with unit
tests at some point. Hopefully soon.

No need for the AS creation callback

My detection of user errors creates some invalid accesses

rename a field for cosmetics

greatly simplify the way peers are created in Vivaldi AS

We used to create a mini cluster for each added peer, which is useless
since they are already in a Vivaldi AS. So instead, add the link
up&down to the Vivaldi containing AS.

This way, we can hide again the netcard of the AsImpl, that was made
public only a few commits ago.

network/coordinates option is  now useless

Detect and diagnose a possible user error

make our FindSimGrid.cmake more visible (and improve it)

please sonar, praise sonar

Use boost instead of xbt in Vivaldi

use NetCard extension to store vivaldi coordinates

make NetCards extendable, and use this for NS3 extension

cosmetics

use std::vector instead of xbt_dynar_t (in NS3)

This should fix quite a lot of leaks, too.

NS3 extensions are for netcards, not for hosts

We need to have it for every netcard, be it an host, an AS or an
host-less router.

It's cumbersome to put it onto hosts when possible and onto netcard
for routers and ASes, and then search first in the host and if not
found in the netcards.

Instead, only put it on netcards, and we're set.

move the NetCard related functions into a NetCard.cpp

actually, BypassRoute can be made private to AsImpl

rename AsRoute to BypassRoute, and move it to its own file

Kill Onelink now that NS3 does not use it anymore

make an As::onRouteCreation signal so that NS3 does not need getOneLinkRoutes afterward

kill the last remainings of RoutingPlatf