[Rock-dev] <base/motion_command.h>

[Sylvain Joyeux]

On 11/07/2011 03:36 PM, Matthias Goldhoorn wrote:
> Please add (if not availible) an constructor and set the time to zero on
> the initialization.
No need since Time initializes itself to a null time by default.

> What's with the other motion commands like AUVPositionCommand and
> MotionCommand2D, from my point of view there should be also an timestamp.
>
> Should it be added to all MotionCommands to be consistent?
I agree in principle. However, these types are very widely used and we 
are in a release process, so do NOT change things unless it is 
absolutely required. Which might be the case for AUVMotionCommand.

In any case, I'd like you to wait for Alex to come back so that he can 
give his opinion. Or (even better) a more in-depth discussion on the 
long-term future of these types which are completely ad-hoc right now.

-- 
Sylvain Joyeux
Space&  Security Robotics

!!! Achtung, neue Telefonnummer!!!

Standort Bremen:
DFKI GmbH
Robotics Innovation Center
Robert-Hooke-Straße 5
28359 Bremen, Germany

Phone: +49 (0)421 178-454136
Fax:   +49 (0)421 218-454150
E-Mail: robotik at dfki.de

Weitere Informationen: http://www.dfki.de/robotik
-----------------------------------------------------------------------
Deutsches Forschungszentrum fuer Kuenstliche Intelligenz GmbH
Firmensitz: Trippstadter Straße 122, D-67663 Kaiserslautern
Geschaeftsfuehrung: Prof. Dr. Dr. h.c. mult. Wolfgang Wahlster
(Vorsitzender) Dr. Walter Olthoff
Vorsitzender des Aufsichtsrats: Prof. Dr. h.c. Hans A. Aukes
Amtsgericht Kaiserslautern, HRB 2313
Sitz der Gesellschaft: Kaiserslautern (HRB 2313)
USt-Id.Nr.:    DE 148646973
Steuernummer:  19/673/0060/3
-----------------------------------------------------------------------