Factorize code for xbt_dynar_shrink/_xbt_dynar_expand.

There's no point in clearing mem in _xbt_dynar_expand.

It's already done in _xbt_dynar_set_at_ptr.

Kill dead (commented) code.

Factorize code for xbt_dynar_set/xbt_dynar_set_at_ptr.

Unlock dynar at proper place.

Don't call _xbt_dynar_expand when it's not needed.

Don't clear mem before free.

Use a tool like valgrind to check for memory errors.

Push last_route in global_routing.

Now we are able to free it at exit.

Need to have my own link list.

Else, it is freed while I'm still using it.

Free cost table on finalize.

Strings are already dup'ed by caller.

memcheck_tests.cmake: regenerate

Set path for binary in token ring tesh file.

xbt_dict_new_ext() doesn't exist anymore.

Fix warning about clobbered variable.

Fixup for commit 08ac8cf2900.

Looks like I forgot this changed line.

make the string large enough to avoid buffer overflows

get workstation model to pick the right network model, the one given as default one (done the crude way, and LV08 actually breaks some stuff)

there is no route in vivaldi

Add a test for model RuleBased.
Cause now in cluster it is model "cluster".

Lua: change the return values of simgrid.task.send.

Also fix a bug in checking the comm type.

MSG_comm_test and MSG_comm_testany did not update isused

Lua: debug chord again

Regenerated memcheck tests.

Lua: fixing the chord example

Lua: simplify the return values of simgrid.comm.test()

Lua: implement isend, dsend and irecv

Lua: fix Chord example

Nobody uses MSG_get_errno and wants useless calls to SIMIX_process_self

SIMIX: add a redundant parameter to avoid repeated SIMIX_process_self()

Remove a useless function. The default one does the same thing.

plug some more memleaks. We have no leak on the GOAL example anymore, yuhu

Add a platform with a bypass route. Add the test to cmake.
Update flatifier and basic_parsing_test to not put "\t" but spaces.

forgot to recreate the option when reviving supernovae

revert [a30523e] (remove supernovae) because I need some timings on G5K, and fix what broke in supernovae mode since then

plug some more memleaks. aint bad anymore

Lua: removing the MSG_process_sleep hack changes the order of output

Lua: copy tasks so that the sender doesn't have to sleep

This will allow to implement asynchronous comms (isend/irecv)

Cosmetics

plug a bunch of memleaks

damn it, there is no example testing the bypass mecanism!!

replace a new/free cycle by a locally stacked object

work to enforce that link arg is readily usable in r_g_r_a_l

reduce the dynar creation madness in g_r_a_l in recursive case

rewrite an error message

merge get_route and get_latency in routings too

rename a global to avoid namespace pollution (and other cosmetics)

kill useless asserts: this is not the way routing logics will take to notice about non-existant routes

further streamline r_get_route_a_l: now that latency is nicely additive, no need for extra precaution

simplify _routing_get_route_and_latency wrt latency

- remove duplicated asserts about non negative latency
  routing_get_route_and_latency (w/o _) checks it
- give the responsability to initialize it to the caller
- streamline the code

Change again the calling convention of routing_get_route_and_latency

now, you cannot call it without passing a valid route receiver. It
will be created if pointing to NULL, or simply filled. Actually that
was the de facto calling convention in the code.

This allows to simplify routing_get_route_and_latency by not checking
if the caller is interested in the route at every line.

(plus some useless cosmetics)

check that AS->get_route don't return NULL on non-existant route, but throw an exception

kill the now useless 'cleanup' parameter of routing_get_route_and_latency

Instead, if route == NULL we won't bother collecting the route, so
freeing is useless.
If *route == NULL, we create a short-lived dynar (equivalent to the
result of having cleanup=1 previously
else, we fill that route without messing with it

kill global_routing->get_route_no_cleanup()

use routing_get_route_and_latency(....., 0) instead

kill routing_get_route(), use routing_get_route_and_latency() instead

AS->get_route is no longer in charge of creating the route it fills

The whole idea is to reduce the dynar creation, copy, destroy madness
in the router. This step is only the first one on that way.

It may well happen that this change leads to massive memleaks. I'll
check after this later.

simplification + add comments + reindent of _get_route_and_latency

useless cosmetics

pre-parse the vivaldi coordinates

another empty function murder

nah, properties in the XML file are not the right place to pass configuration options

Actually, the properties are the right location, but only when
included in a config tag. And in this setting, the right behavior is
already done by the </config> handler, no need to also do it for the
generic <prop> handler. Doing so anyway would prevent to pass
"coordinates" (user-)properties to hosts, links and such.

don't trust globals to be memset'ed to 0 (and rework a message)

error message rewrite

Let's trust the mathematics here: sqrt + fabs + fabs IS positive or nul :)

tiny cleanups, and add a FIXME

merge structures route_t and route_extended_t.

All this added complexity to save 2 pointers was a bit too much for
me. Yeah, YMMV, but I prefer it this way.

tiny function renaming

fix the breakages that were hidden because some tests were disabled

pcre is not optional anymore since a while -- reactivate the vivaldi tests

Lua: implementing asynchronous comms (wait, test, isend, dsend, irecv)

Lua: add a function simgrid.process.sleep()

Lua: add a function simgrid.get_clock()

Lua: document the possibility to use numbers for mailboxes

Lua: return errors from execute/send/recv, or nil in case of success

Lua: rename simgrid.task.name() to simgrid.task.get_name()

Lua Chord example (not working yet, the API is still incomplete)

Make MSG_TASK_CANCELLED deprecated, replace it by MSG_TASK_CANCELED

Remove ruby example from simgrid.

Use xbt_XXX_is_empty() instead of testing xbt_XXX_length() against 0.

Where XXX is 'dict' or 'dynar'.

(was a request from emptty)

Rewrite xbt libs, using xbt dicts.

And remove lots of code duplication.

Also remove unused functions.

Use xbt_lib_length instead of xbt_lib_size.

Use defined interface for accessing to lib properties.

Remove usage of xbt_dict_size().

xbt_dict_size() can now be safely removed.

Remove trailing spaces in ChangeLog.

Added the new logo to the unstable doc, so that Lucas and Fred can show a webpage and at least one doc with the same logo at SC

kill another "method" away from the "singleton" global_routing

OO coding in C is something, but applying the worst possible OO design
to C is ... inventive

cosmetics and improve doc

kill a file that were begging for it

cosmetics

move the callbacks to the cluster creation from surfxml_parse to sg_platf

I've found the name of the next release

move some XML specific code from surf_routing.c to surfxml_parse.c

Improve a bit the documentation of hte xbt_queues, and ensure that they are displayed on the web site

Remove two more declaration of phantom functions.

Simply say fun_ptr(...) instead of (*fun_ptr)(...).

Another possible cleanup in the same spirit is to remove the
ampersand when taking the address of a function.
This is left as an exercise for the reader.

some more cleanups for tomorrow marked as FIXME for now -- good night

various psychorigid cleanups -- routing_parse_cluster() is now readable

that's a quite complicated no-op :)

Merge branch 'parser'

do not search the cluster backbones again and again, store them in the right location

On the poor tests I've done, it seems to give a non negligible
improvement on the goal example. Maybe 5-10%, but it's hard to say
right now.