Olivier Stasse authored 3 years ago and Olivier Stasse committed 3 years ago

While here fix the exception message for int (remove the second int).
Fix the test accordingly.

Olivier Stasse authored 3 years ago and Olivier Stasse committed 3 years ago

Add support for long int.

fix:

…/dynamic-graph/src/dgraph/entity.cpp: In member function ‘void dynamicgraph::Entity::sendMsg(const string&, dynamicgraph::MsgType, const string&)’:
…/dynamic-graph/src/dgraph/entity.cpp:212:52: warning: ‘void dynamicgraph::Logger::sendMsg(std::__cxx11::string, dynamicgraph::MsgType, const string&)’ is deprecated: use stream(type, lineId) << msg [-Wdeprecated-declarations]
   logger_.sendMsg("[" + name + "]" + msg, t, lineId);
                                                    ^
In file included from …/dynamic-graph/include/dynamic-graph/entity.h:18:0,
                 from …/dynamic-graph/src/dgraph/entity.cpp:13:
…/dynamic-graph/include/dynamic-graph/logger.h:225:3: note: declared here
   sendMsg(std::string msg, MsgType type, const std::string &lineId = "");
   ^~~~~~~

ref. #58

This reverts commit 62765608.
as this is breaking on boost < 1.73

to fix debian packages creation on ROS buildfarm:

00:12:11 dpkg-shlibdeps: error: cannot find library libtracer.so.4.3.1 needed by
debian/ros-noetic-dynamic-graph/opt/ros/noetic/lib/dynamic-graph-plugins/libtracer-real-time.so.4.3.1
(ELF format: 'elf64-x86-64' abi: '0201003e00000000'; RPATH: '')

   "[signal-cast-helper] Removed unused template specialization".
   DefaultCastRegisterer<std::string>::cast template specialization is
   required to set empty string to a signal.

This commit removes template specialization not exposed in header
files, and better handled by the default implementation.

Olivier Stasse authored 5 years ago and olivier stasse committed 5 years ago

Based on feedback from G. Saurel.

  - when signal value contained a space, only the part before the space
    was stored in the signal.
  - add a test on this case.

  Default implementation adds std::endl at the end of the stream.
  In the case of string this extra '\n' is not desired.

  When type is string and input istringstream contains an empty string,
  operator>> make method fail return true.
  This commit overcome this issue by not testing the result.