Merge branch 'fix_bindings_mistakenly_set_as_member_functions' into 'master'

Make Python bindings static where needed

See merge request simgrid/simgrid!83

diff --combined src/bindings/python/simgrid_python.cpp
index f79cfb0,b6f5e05..0b1721b
--- 1/src/bindings/python/simgrid_python.cpp
--- 2/src/bindings/python/simgrid_python.cpp
+++ b/src/bindings/python/simgrid_python.cpp
@@@ -24,6 -24,7 +24,6 @@@
  
  #include "simgrid/kernel/ProfileBuilder.hpp"
  #include "simgrid/kernel/routing/NetPoint.hpp"
 -#include "src/kernel/context/Context.hpp"
  #include <simgrid/Exception.hpp>
  #include <simgrid/s4u/Actor.hpp>
  #include <simgrid/s4u/Comm.hpp>
@@@ -317,7 -318,7 +317,7 @@@ PYBIND11_MODULE(simgrid, m
    /* Class Host */
    py::class_<simgrid::s4u::Host, std::unique_ptr<Host, py::nodelete>> host(
        m, "Host", "Simulated host. See the C++ documentation for details.");
-   host.def("by_name", &Host::by_name, "Retrieves a host from its name, or die")
+   host.def_static("by_name", &Host::by_name, py::arg("name"), "Retrieves a host from its name, or die")
        .def(
            "route_to",
            [](const simgrid::s4u::Host* h, const simgrid::s4u::Host* to) {
@@@ -423,7 -424,7 +423,7 @@@
              h->set_pstate(i);
            },
            "The current pstate (read/write property).")
-       .def("current", &Host::current, py::call_guard<py::gil_scoped_release>(),
+       .def_static("current", &Host::current, py::call_guard<py::gil_scoped_release>(),
             "Retrieves the host on which the running actor is located.")
        .def_property_readonly(
            "name",
@@@ -442,7 -443,7 +442,7 @@@
                               "Get the available speed ratio, between 0 and 1.\n"
                               "This accounts for external load (see :py:func:`set_speed_profile() "
                               "<simgrid.Host.set_speed_profile>`) (read-only property).")
-       .def(
+       .def_static(
            "on_creation_cb",
            [](py::object cb) {
              Host::on_creation_cb([cb](Host& h) {
@@@ -626,7 -627,10 +626,10 @@@
        .def(
            "__str__", [](const Mailbox* self) { return std::string("Mailbox(") + self->get_cname() + ")"; },
            "Textual representation of the Mailbox`")
-       .def("by_name", &Mailbox::by_name, py::call_guard<py::gil_scoped_release>(), "Retrieve a Mailbox from its name")
+       .def_static("by_name", &Mailbox::by_name,
+                   py::call_guard<py::gil_scoped_release>(),
+                   py::arg("name"),
+                   "Retrieve a Mailbox from its name")
        .def_property_readonly(
            "name",
            [](const Mailbox* self) {
@@@ -811,7 -815,7 +814,7 @@@
        .def_property_readonly("pid", &Actor::get_pid, "The PID (unique identifier) of this actor (read-only property).")
        .def_property_readonly("ppid", &Actor::get_ppid,
                               "The PID (unique identifier) of the actor that created this one (read-only property).")
-       .def("by_pid", &Actor::by_pid, "Retrieve an actor by its PID")
+       .def_static("by_pid", &Actor::by_pid, py::arg("pid"), "Retrieve an actor by its PID")
        .def("set_auto_restart", &Actor::set_auto_restart, py::call_guard<py::gil_scoped_release>(),
             "Specify whether the actor shall restart when its host reboots.")
        .def("daemonize", &Actor::daemonize, py::call_guard<py::gil_scoped_release>(),