jawi / ols

OpenBench LogicSniffer repository
http://www.lxtreme.nl/ols/
319 stars 75 forks source link

Setting baud rate no longer works with next branch #197

Closed matthijskooijman closed 3 years ago

matthijskooijman commented 11 years ago

In a recent commit of the next branch, the parsing of the baud rate stopped using NumberUtils.smartParseInt and is just using Integer.parseInt now. However, the values in the dropdown include a "bps" suffix, which was stripped by smartParseInt, but now causes parseInt to throw an exception and a baudrate of "null" to be inserted into the connection string.

Not sure what the proper way to fix this is, though.

matthijskooijman commented 11 years ago

Note that this bug broke communication with my Bus Pirate. However, if I just manually enter 115200 without the bps postfix, everything works as expected.

jawi commented 11 years ago

Thanks for reporting this, I already found this issue myself, but you beat me to reporting it ;-)