Dropped MODULE_PRE_START macro, and welcomed a new module_pre_start i…

…nterface function declared by MODULE macro.

Lib/module.h

@@ -27,6 +27,7 @@
 
 /* Interface Macros */
 #define MODULE_CTX(name, ctx) \
+    static void _ctor1_ module_pre_start(void); \
     static int init(void); \
     static int check(void); \
     static int evaluate(void); \
@@ -42,8 +43,6 @@
     static void _dtor1_ destructor(void) { module_deregister(&self); }
 
 #define MODULE(name) MODULE_CTX(name, DEFAULT_CTX)
-
-#define MODULE_PRE_START() static void _ctor1_ module_pre_start(void)
 
 /* Defines for easy API (with no need bothering with both self and ctx) */
 #define m_is(x)                     module_is(self, x)

Samples/Easy/a.c

@@ -16,7 +16,7 @@ static void recv_ready(const msg_t *msg, const void *userdata);
  * called before registering the module. Use this to set some 
  * global state needed eg: in check() function 
  */
-MODULE_PRE_START() {
+static void module_pre_start(void) {
     printf("A: Not yet inited!\n");
 }
 

Samples/Easy/b.c

@@ -14,7 +14,7 @@ MODULE("B");
  * called before registering the module. Use this to set some 
  * global state needed eg: in check() function
  */
-MODULE_PRE_START() {
+static void module_pre_start(void) {
     printf("B: Not yet inited!\n");
 }
 

docs/src/callbacks.rst

@@ -5,17 +5,23 @@
 Module callbacks
 ================
 
-Every module needs 5 callbacks that be must defined by developer. |br|
-They are automatically declared by MODULE macro:
+Every module needs 5 functions that be must defined by developer. |br|
+They are automatically declared by MODULE macro. |br|
+Moreover, a module_pre_start function is declared too, but it is not needed by libmodule interface, ie: it can be left undefined.
 
 .. code::
-    
+    static void module_pre_start(void);
     static int init(void);
     static int check(void);
     static int evaluate(void);
     static void recv(const msg_t *msg, const void *userdata);
     static void destroy(void);
 
+.. c:function:: module_pre_start(void)
+
+  It can be used to create a function that will be called before any module is registered. |br|
+  It is the per-module version of :ref:`modules_pre_start <modules_pre_start>` function. |br|
+
 .. c:function:: init(void)
 
   Initializes module state

docs/src/module.rst

@@ -15,12 +15,6 @@ These macros make it easy and transparent to developer all of the module's inter
 It enforces correct modularity too: each module must have its own source file. |br|
 Where not specified, these functions return a :ref:`module_ret_code <module_ret_code>`.
 
-.. c:macro:: MODULE_PRE_START()
-
-  This macro can be used to create a function that will be automatically called before any module is registered.
-  It is the per-module version of :ref:`modules_pre_start <modules_pre_start>` function.
-  It does not return anything.
-
 .. c:macro:: m_is(state)
 
   Check current module's state