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

cosmetics

less VM in surf.h

Kill the fact that several model VM models may be provided.

- VM are a plugin: take it, copy it and change it yourself
- helps removing VMs from the trunk (I demand you to enter this plugin!)

please sonar and use & when creating function pointers

cosmetics

move NetCard to its own header file

no need for a NetCard and a NetCardImpl when both are in the same file

add -no-pie flag to thread sanitizer (see https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=796246)

please my old friend SonarQube now that it's back

SonarQube: upgrade scanner to the latest version 2.8

2.6.1 fixes https://jira.sonarsource.com/browse/SQSCANNER-18
so that warning message are properly displayed in log.

SonarQube: remove useless properties "sonar.c.std" and "sonar.cpp.std"

When using build-wrapper this is inferred based on an actual build.

please sonar: this variable will be overwritten in the next loop cycle

try to fix a shell syntax error, bummer

fix the syntax in the sonar configuration (fix #126)

I drop the #126 PR that propose to kill that config item all together
because I really think that sonar should not feel too much concerned
by our logging macros.

Yes they increase the nesting level if you expend them, but nobody
expends them mentaly. We use them as inline functions even if we
cannot implement them as functions for technical reasons.

Don't show the sonar token in the logs (fix #127)

the token was regenerated, too.

kill the now empty RoutingPlatf datatype \o/

move the rootAS from routing_platf into the EngineImpl

Now that we instanciate a s4u::Engine when using the routing, the
surf_usage tests don't have to manually call surf_exit(): it called
atexit() during initialization. Gosh, it took me so long to figure out
why these two tests would break when the rootAs is moved...

tiny optim

Convert routing_platf->getRouteAndLatency into s4u::Host->routeTo

The bad thing is that the public API does not expose the surf::Link
type, so there is little you can do about it. For now.

don't really use netCards in the network model. Keep at the host level

fix java build

cleanups in the dict and dynar tests

stop testing heterogeneous dicts, too

Deprecate xbt_dict_new()

Please people, use xbt_dict_new_homogeneous(). Non-homogeneous dicts
were a very bad idea that could have only originated in Perl anyway.

don't pass the free_f parameter to MSG_*_set_property_value()

Don't store raw objects as host properties, which must be strings

cosmetics

update a teshfile that was affected by #123, which Fabien just fixed

call updateRemaining when the action is suspended (fix #123)

if not, the LMM is giving the right results, but surf does not notice.

Thus the fix does not touch LMM, only surf_interface.cpp. Note that
suspend has the specificity that we cannot estimate when it will end.
Thus it does not seem appropriate to use the nextOccuringEventLazy
method which is essentially trying to find when will be the next step.
Instead I propose to call updateReaminingLazy from the suspend
function.

make Boost.test optional, as I fail to find it on appveyor

plug an obvious memleak, bummer

Add boost unit_test_framework to our dependencies

Merge branch 'master' of github.com:simgrid/simgrid

Merge branch 'master' of scm.gforge.inria.fr:/gitroot/simgrid/simgrid

cosmetics

more informative assert message

add an EngineImpl, unused so far

cosmetics (and stop using routing_platf)

gosh, so much cruft in this code

Merge pull request #124 from mpoquet/master

[Energy platform parsing] assert + memory leak

Merge branch 'master' of https://github.com/mpoquet/simgrid

Energy: Fixed host parsing assertion + mem leak

When parsing the platform file with energy,
using the old watt description syntax
"watt_min:watt_max" instead of
"watt_idle:watt_min:watt_max" led to obscure
OOB error, the error message should now be
clearer.

Also found a little memory leak,
msg_idle was not freed.

disable java for MC FreeBSD builds, as it breaks mmalloc

On Freebsd with clang > 3.8, sometimes we have a null name here.
Try to not segfault in this case.

for some reason (const madness?) this lead to wrong names being printed.

Switch the event handling in the MC to libevent.

This is a good alternative to provide a clean socket/signal
handling, which allows to finaly enable the MC on FreeBSD too. The
downside is that now libevent>=2 is necessary to build the model
checker.

Thanks to degomme@ for the mentoring ;)

Add more libraries to blacklist.

FreeBSD system and LLVM related libraries.

BSD/SysV ptrace resume addr is 1, not 0 like in linux.

improve the debug logs produced by the route seeking mechanism

remove a ugly misplaced include

Merge branch 'master' of scm.gforge.inria.fr:/gitroot/simgrid/simgrid

Fix umpire issues : MPI_Comm cannot be safely put inside an int
Fix/Ignore some warnings as well

More ptrace portability in model checker.

No PTRACE_O_TRACEEXIT equivalent in the original ptrace API, futher
work will indicate if this will be a problem.

Update message a bit when neither linux's or BSD's version are
available.

Mark shared libraries private clean pages as non-writable on FreeBSD.

This is how persmissions are with Linux, and later simgrid identifies
mappings based on path and permissions, so the less restistance path
is to present things more linux-like.

In FreeBSD map backend, mark stack same as in Linux

Add support to retrieve FreeBSD ucontext registers.

Sadly, no universal method as it depend on the CPU and the system
is free to implement whatever storage it see fit. Leave a terminating
error message for the next system to get ported.

Add alternative to remap() in PageStore when not available.

Code is a bit simplistic, having way to expand mappings but not
shrink them (so leaving them as they are), but it does the job.

Set waitpid() for model-checked flags according to platform.

use the right name of an option in the doc

fix lua build

move VM into their namespace, and greatly reduce the amount of sg_host_get_name (prefer C++)

inline yet another function in the VM

vm: kill some useless static functions

inline 4 more VM simcalls

I had to add some methods to s4u::VM, such as getRamsize()

VM: improve error messages, and prefer C++ over MSG

VM: keep inlining dummy functions

I could merge my commits, but these changes are really hard to review
so don't make it more complex than needed

VM: inline a function

VM: code simplification. Much more needed :(

give s4u::Host a cname() that returns a char*

kill commented code, anonymous blocs and reindent. No real change