Ok, I just ran through this with Adam. It's now time to open an API review on rosserial: <a href="http://www.ros.org/wiki/rosserial/Reviews/2011-08-04">http://www.ros.org/wiki/rosserial/Reviews/2011-08-04</a> API Review_API_Review (I'll be sending out an official announcement shortly) <br>
<br>As to your patches:<br><br>1) Allow custom baudrates -- I've added this to our API review discussion. The issue becomes the PC-side, in that we don't want to have *any* configuration data (and that includes baud rate) stored on the PC. <br>
<br>2) Simplify Subscriber/Publisher constructors -- we actually don't want to do this. While we currently don't make use of the message pointer, there is some automatic packet resend work in development that would make use of the message pointer. If you still feel this is necessary, please bring it up in the API review. <br>
<br>3) Signed char* -- good catch, I created a ticket (#20)<br><br>I'm posting comments to most of your tickets, Adam is taking a look at #18 (CMake issues). <br><br>-Fergs<br><br><br><br><br><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
Attached are a couple of patches I'd ask for inclusion into rosserial:<br>
<br>
- allow custom baudrates<br>
- simplify Subscriber/Publisher constructors<br>
- make strings signed char*, instead of unsigned char*<br>
<br>
cheers<br>
Dariush<br><font color="#888888">
-- <br>
Institut für Technische Informatik / Universität zu Lübeck<br>
<a href="http://www.iti.uni-luebeck.de/index.php?id=forouher0" target="_blank">http://www.iti.uni-luebeck.de/<u></u>index.php?id=forouher0</a><br>
</font><br>_______________________________________________<br>
ros-users mailing list<br>
<a href="mailto:ros-users@code.ros.org" target="_blank">ros-users@code.ros.org</a><br>
<a href="https://code.ros.org/mailman/listinfo/ros-users" target="_blank">https://code.ros.org/mailman/listinfo/ros-users</a><br>
<br></blockquote></div><br>