Don't define routing_component_t twice.

Yet another clobbered variable warning

Clean the code and comments of registering C functions to Lua

These function don't return anything

Move this debugging function to lua_utils.c

The argument is a userdata, not a Lua task

Lua: merge simgrid.task.recv and simgrid.task.recv_with_timeout

The timeout is now an optional argument of simgrid.task.recv.

Document C functions that Lua calls

When a C function is called by Lua, get the args with positive indices

A negative index is unsafe since the caller may provide more arguments
than needed.

Nice joke.

Make the Lua API respect coherent naming conventions

Rename functions in simgrid_lua.c

Remove a useless xbt_assert from the critical path

The error message was not more informative than a segfault will be,
and user behavior cannot produce this (I think)

use a sane way to declare postparse callbacks, now that this way exists

partially reindent that file

Remove the complexity needed to save a global that is never used

this is the field "model" in the surf_model_description structure. The
actual model is always saved elsewhere (like in the global
surf_workstation_model and friends).

If one day we want to have multi-model simulations, we'll need to
partially revert this change, but I guess that this day seaching the
right model linearly by name, as it was made possible with the
existing setup, will not be the way to go.

I don't see the point of declaring and calling empty functions

*_load *_unload callbacks of the routing logics seem unused.
We could maybe kill them?

oups, that comment were in french

nobody use these structures, so I won't try to understand their purpose

Reorganize mallocator.c, and add the possibility to disable mallocators.

Memory leaks are sometimes hard to track when mallocators are in use.
Set MALLOCATOR_IS_WANTED to 0 to disable them during your debugging session.

The code for mallocator_new and mallocator_get is also made a bit cleaner.

Kill "if" before free, and useless cast of free argument.

free(NULL) is a no-op, there is no need to test the given pointer
before call.
The same argument stands for xbt_free, xbt_dict_free, xbt_dynar_free,
and xbt_strbuff_free.

Replace, when possible, calls to bprintf() by calls to xbt_strdup().

For radical it is "," and not ";"

improve documentation of MSG_task_dsend function

Also remove the comm from processes when SIMIX_post_comm is not called

Fix possible crashes and leaks with dsends during processes cleanup

Destroy sg_platf_link_cb_list on sg_platf_exit().

Don't duplicate strings uselessly, and avoid memory leaks.

Reindent before change.

Reindent surf_routing.c

Constification of parameter, and attribute no_return.

Add a script running GNU indent with the appropriate options.

The set of options was given by README.coding which is also updated.

Factorize routing_parse_cluster.

I demand you to stop that empty function chaining madness. NOW

sanitize the names of the routing's parse callbacks

no need to have the routing_parse_Scluster() function public. routing_register_callbacks is enough

sanitize the way parsing callbacks of the routing submodule are registered

One day, we'll have to deal with route creation in sg_platf. That day, we'll need that data structure

simplify this function

rename some types for consistency

that FIXME would be a bad idea to implement

reindent, no real change

implement cluster tag on top of sg_platf w/o XML black magic

Kill the XML-level callbacks to AS_open/close and create some at sg_platf level instead

simplify that vicious nasty code

kill dead code

/me also hates void* pointers

/me hates globals

integrate the peer creation in sg_platf properly

preparse the arguments of peer tag

improve peer creator: use sg_platf_new_*, not the XML bypassing mechanism

Destroy dax parser after use.

Fix memory leak in xbt_cfg_set_parse().

Kill bogus fixme: xbt_free_f already exists.

Parenthesis should be protected in test parameters.

further parser cleanups

- kill some dead code related to the old lua console
- do not load a header at the *end* of another header
- solve some header inclusion madness created in this process

do use the parameters I'm given, not some random globals

kill dead code

improve comments

plug a memleak when the cluster's router_id is not given

fix constness in parser

further fields renamings in the parser

Update README of Lua examples

Clean the code of Lua examples.

Reindent, remove useless semicolons, declare variables as local, etc.

Add unversioned chord large deployment files to .gitignore

Also remove '\r' characters from .gitignore

These files have nothing to do here

further code simplification. This one were nasty

simplify the code a bit

Also cleanup the fields of s_sg_platf_link_cbarg_t

This one is not just a renaming:
 - kill the unused V_policy_initial_link
 - make V_link_sharing_policy (now 'policy') a
   e_surf_link_sharing_policy_t (which thus become public)
This should be enough for field renaming for now. But the cleanup is
definitly not finished :-/

also cleanup the field names of s_sg_platf_router_cbarg_t

Still manual, so atomic commits to avoid issues

Parser cleanup: simplify structure's fields' names

Damn, my eclipse fail that refactoring! That's manual, pity me...

Reimplement the lua console on top of sg_platf

I admit it was too early to do that change since sg_platf is not quite
fully separated from the XML parsing yet, but the cruft accumulates
from everywhere, so I have to break stuff a bit to fix the overall
picture. Yeah, the lua tests are not fixed yet.

kill more empty function chaining in lua console

Slight cleanups in the lua console

- kill some empty functions chaining
- allow for the creation of routers from lua
(the console is still broken, more profund cleanups will be needed)

Kill some further empty function chaining.

This breaks the lua console for a while, but this is to be redone with
the new sg_platf anyway.

Properly integrate the link creation into sg_platf

This structure was not necessary

Lua: don't garbage collect a task after sending it (the receiver will)

Also clean the code of Lua tasks. Show an appropriate error message when
the sender tries to access a task already sent to someone else.

Lua: add a function sglua_copy_value

Add comments about who should or shouldn't call MSG_task_destroy

After a send, the sender should never access the task anymore.

Since processes are in different Lua worlds, the receiver can garbage
collect the task.

further parser cleanups

Properly integrate the begin and end of platform definition in sg_platf.

New public functions:
 - sg_platf_open()
 - sg_platf_close()
New internal callback:
 - sg_platf_postparse_add_cb(void_f_void_t)
 (nobody cares about sg_platf_open(), it's just here for cosmetics in
  user code)

also deprecate ETag_surfxml_router

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

Further parser cleanups

- Kill useless function chaining
- Deprecate the ETag_host in XML now that it's useless

Don't use a fix value of host.

Add a source file for model vivaldi. Write vivaldi_get_route.
Update vivaldi platforms (examples/platforms/syscoord/

Remove some bprintf, fix memleaks.

fix the last broken tests by sanitizing the way properties are handled during the parse

Resolved merge "merge changes. Strange, it seems to ate pierre commit" and
 redo the missing part. Set the struct for get_route of tag cluster.

Remove unused function.

Further parser cleanups

Simply rename a bunch of symbols to make the module logic clear

integrate properly the router creation in sg_platf module

plug memleaks: surfxml does not allocate memory, sg_platf (and callbacks) do it on need

Make the sg_plat_new_host() function public

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

Continue parser cleanups

Prepare to split the interface of sg_platf in two: one public
interface (include/simgrid/platf.h) will contain the function
available to the user code to create elements while another interface
(src/include/simgrid/platf_interface.h) will contain what internal
modules need to declare callbacks on the element creation events.

For now, the files are still very empty, but I try to not break stuff
even further, so I commit soon.

The parser's behavior has changed, revalidate the output of msg-trace

Add some _XBT_GNUC_UNUSED to avoid compilation warnings with NDEBUG

This include is not needed anymore.