Enhancements in SoftwareSerial lib

[sebastienjean]

I tried to use SotfwareSerial to communicate with a device at 600 Baud. Unfortunately, i did not succeed. And after spending some time to figure out the failure, i noticed that

• the 600 Baud speed was simply not supported (no delays in DELAY_TABLE)
• begin(long speed) was returning normally even if speed was not supported.

So, i have added a 600 Baud line in DELAY_TABLE for each CPU frequency. This has been fully tested for 16 MHz. I have also changed the return type of begin(long speed) from void to bool in order to notify unsupported speeds.

[cmaglie]

Thanks,

can you repost the patch without the change to begin() function? The supported speeds are documented in the reference page, I don't feel the need to have a notification returned from begin.
The point is: even knowing at runtime that the speed is not supported, what you can do afterward (besides halting)?

http://arduino.cc/en/Serial/Begin

[sebastienjean]

Done.
I thought a non-void returning type could have been useful at init time when associated to a notification using a LED.
Anyway, one can simply RTFM (as i did ;-) ).

[cmaglie]

Great! Thank you.

[sebastienjean]

Thanks are for you.
How can the reference be in sync with the code, in order to mention 600 Baud as a supported rate ?

[cmaglie]

Just updated also the doc page ;)

[sebastienjean]

But someone has also not to forget this one for the next release
http://arduino.cc/en/Reference/SoftwareSerialBegin

[cmaglie]

Whoops. Ok fixed.