On line 150 of Bee.cpp:

for(int i = 0; i < sizeof packet; i++) throws a compiler warning because sizeof returns a std::size_t which is of type unsigned integer.

Propose changing the declaration in the for loop to type uint16_t:

for(uint16_t i = 0; i < sizeof packet; i++)

Hi kmark,

I'm using a slightly modified version of your library in my own project to provide a reliable streams based (select repeat arq based) protocol. Great library. Thanks.

As the interface to our protocol is a sub class of the Streams class we are essentially exposing a driver (albeit one that encapsulates Hardware or Software Serial).

To take our code to the next level, what would be ideal is to be able to have the actual xbee reads/writes done in interrupt mode.

I have done some crude experiments using an interrupt timer - but calling XBee tick within an interrupt is hazardous/unreliable and may need some better smarts than I have to make work properly.

Welcome your thoughts, and if you were interested in assisting with this.

Andrew

In testing with this library, there was a case where an Xbee packet was sent where the packet size byte was escaped. Looks like the simple hack is to add the isEscaped logic to the packet size switch.

Hi there

We use a slightly modified version of your great library. However, since moving to the SX range of devices we've been having issues with modems getting bricked and so are going to implement rts/cts handshaking.

Have you considered implementing this for your library?

Greetings,

I'm attempting to get this up and running using Arduino 1.6.7 and Teensyduino. Upon compile, I am getting the following errors. It appears as though the Serial construct in the library for one reason or another has a different data type that the HardwareSerial core. Do you happen to have any ideas or insight?

Thanks!

C:\Program Files (x86)\Arduino\libraries\Bee\Bee.cpp: In member function 'void Bee::sendData(String)':
C:\Program Files (x86)\Arduino\libraries\Bee\Bee.cpp:150:31: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
for(int i = 0; i < sizeof packet; i++) {
^
C:\Program Files (x86)\Arduino\libraries\Bee\Bee.cpp: In member function 'void Bee::write(char, uint16_t)':
C:\Program Files (x86)\Arduino\libraries\Bee\Bee.cpp:240:27: error: invalid conversion from 'char_' to 'const uint8_t* {aka const unsigned char_}' [-fpermissive]
serial->write(c, size);
^
In file included from C:\Program Files (x86)\Arduino\hardware\teensy\avr\cores\teensy3/WProgram.h:16:0,
from C:\Program Files (x86)\Arduino\hardware\teensy\avr\cores\teensy3/Arduino.h:1,
from C:\Program Files (x86)\Arduino\libraries\Bee\Bee.h:23,
from C:\Program Files (x86)\Arduino\libraries\Bee\Bee.cpp:20:
C:\Program Files (x86)\Arduino\hardware\teensy\avr\cores\teensy3/HardwareSerial.h:192:17: error: initializing argument 1 of 'virtual size_t HardwareSerial::write(const uint8_t, size_t)' [-fpermissive]
virtual size_t write(const uint8_t *buffer, size_t size)
^
exit status 1
Error compiling.

My sketch is:

include <Bee.h>

void setup() {
// put your setup code here, to run once:

}

void loop() {
// put your main code here, to run repeatedly:

}

Hello!
We really appreciate your great library for Digimesh and had a couple of questions for our case.
We want to be able to send data/parameters and receive them at both ends. Our setup includes two Xbees in API 2 mode connected to their respective Arduino mini's. We were wondering when and what the callBack function does? and what we should be expecting to be written to the serial?
To check if we even enter the void function, we each added a println msg there. while one of the arduinos sent a message and confirmed that it had gone in to the callback method, the other one just sent its own msg and didn't enter its callback method.

Anything will be greatly appreciated :)
Thanks in advance