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.

you have to plug the memleaks from time to time

populate the routing_none_finalize to make sure that we always call these functions

cosmetics: variable&fields renaming

Use the none routing as an ancestor to the generic one, which is ancestor to most of the others. First step to further factorization

cosmetics, nothing serious

New Lua example: Chord (work in progress)

sanitize get_network_element_type()

It were declared as a field of global_routing, and of each and every
routing model. That's a bit much for a simple little function.

kill unused global_routing->get_latency()

Kill global_routing->get_route_or_null

It was used at only one location, so I setup a TRY/CATCH in there
instead

That pointer to function was never given anything but one value

The global_routing value seem to be used to group functions together,
but the fields cannot be given another value.

Of course, I'm gonna kill em all.

factorize the routing_exit functionality, and ensure that no model can forget to call it

kill small crufts here and there

kill further globals

saner pattern use in the trace files

I guess that previously, the patterns were replaced for the first
host, and then reused as is for the next ones. So the traces specified
for the first host were used for any ones.

fix the constness of xbt_strbuff_new_from(), it's too dangerous with previous prototype

various cleanups

amongst other, kill yet another global

ahah. Mort de rire. :-/

various cleanups and clarifications

rename routing_component to as, since that's just an AS at the end of the day

Remove section about java.

Make a sync in a different directory.

Lua masterslave: simplify the code and improve the output

Lua: set a task metatable that allows OO-style syntax

You can now write my_task:send(mailbox), which is just syntaxic sugar
for simgrid.task.send(my_task, mailbox).

Lua: garbage collect the simgrid module (remove simgrid.clean())

Don't protect access to the metatable from Lua.

Lua code can never change the metatable of a userdata anyway
(setmetatable is possible only for a table).
getmetatable is now allowed for tasks and hosts: it's harmless and lets
people extend them.

Clean MSG source file names

inline static functions declared only once, and that were not really improving my execution flow understanding

kill unused static functions

that function is so stupid that it is never called. Let's remove it

kill a type that were exactly the same than another

sanitize type naming between surf models and routing models

use an explicit type name for pointer on fct

tiny partial reindent

Move the SURF model initialization from routing_parse to sg_platf

It's cleaner when the router is just another bunch of callbacks, with
no central point. But it's a bit harder because the sg_platf callbacks
are also used during the parsing of deploy.xml

kill unused callbacks in the routing models

they were (probably) intended to allow the rmodels to add new
callbacks over the elements, but since I cannot remove callbacks in
the new mecanism, and since it wasn't used, I don't see the point.

Rename sg_platf_new_AS_{open,close} to {begin,end} since you cannot reopen them to add stuff afterward (and other similar call renaming)

Remove declaration for inexistent functions.

These functions are declared static, define them static too.

Rename some routing logic fields to make them more explicit (hopefully)

kill asserts which error message is as instructive as the segfault that it hides

cluster routing is simpler than you thought

kill unused function

add a constructor to the generic parsing logic, and use it to factorize code