On 13.01.2014 13:52, Sylvain Joyeux wrote: > On Monday, January 13, 2014 01:30:15 PM Matthias Goldhoorn wrote: >> On 13.01.2014 10:42, Sylvain Joyeux wrote: >>> On Friday, January 10, 2014 05:26:18 PM Matthias Goldhoorn wrote: >>>> Here is the updated patchset >>>> >>>> another patch is on the RTT mailing list regard the blocking behaviour >>> Looks much better for me. >>> >>> Minor comments: >>> - I thought we would keep the setBlaBLa for the operation name, only the >>> >>> "internal method" would be called __orogen_setBlaBla. >> Why we still need the "old" operation?, i removed them to have not twice >> methods for the same. >> The method setBlaBla is kept and called from the __orogen_ bla > My idea was: > operation name: setBlaBla > internal method name: __orogen_setBlaBla > user method name: setBlaBla > Then i need to change the orogen implementation a bit more, because as i see in the current design, the operation name equals the methodname. Sinc'e i need the internal function different than the user one this was the most nice solution.
So should i _really_ change orogen to provide another methodname than the operationname. From my point of view makes this again more magic in background?! Matthias -- Dipl.-Inf. Matthias Goldhoorn Space and Underwater Robotic Universität Bremen FB 3 - Mathematik und Informatik AG Robotik Robert-Hooke-Straße 1 28359 Bremen, Germany Zentrale: +49 421 178 45-6611 Besuchsadresse der Nebengeschäftstelle: Robert-Hooke-Straße 5 28359 Bremen, Germany Tel.: +49 421 178 45-4193 Empfang: +49 421 178 45-6600 Fax: +49 421 178 45-4150 E-Mail: [email protected] Weitere Informationen: http://www.informatik.uni-bremen.de/robotik _______________________________________________ Rock-dev mailing list [email protected] http://www.dfki.de/mailman/cgi-bin/listinfo/rock-dev
