address review comments

maliput · Jan 2, 2018 · 57dea01 · 57dea01
1 parent 06edbb0
commit 57dea01
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 5 deletions.
diff --git a/backend/python_bindings_example.py b/backend/python_bindings_example.py
@@ -66,10 +66,7 @@ def get_from_env_or_fail(var):
 
     # Since it is an environment variable, the very end may have a colon;
     # strip it here
-    if value[-1] == ':':
-        value = value[:-1]
-
-    return value
+    return value.rstrip(':')
 
 
 class SimulationStats(object):

diff --git a/backend/simulation_runner_py.cc b/backend/simulation_runner_py.cc
@@ -57,7 +57,7 @@ PYBIND11_MODULE(simulation_runner_py, m) {
     .def("Stop", &SimulatorRunner::Stop)
     .def("AddStepCallback", &SimulatorRunner::AddStepCallback);
   ;
-  py::class_<AutomotiveSimulator<double>, std::unique_ptr<AutomotiveSimulator<double>>>(m, "AutomotiveSimulator")
+  py::class_<AutomotiveSimulator<double>>(m, "AutomotiveSimulator")
     .def(py::init([](void) {
         return std::make_unique<AutomotiveSimulator<double>>();
     }))