Define all of XBeeAddress64 in XBee.h

Since this is a utility class that should really be a very thin wrapper
around the actual data, the compiler should be able to inline all
operations, instead of having to do a full function call just to load or
store a few bytes.

This removes the explicit super-constructor calls and rewrites the
constructors to use initializer lists instead of assigning values in the
body (which is more consise, and allows for constexpr next)

This saves up to a 30 bytes of program spaces on the supplied
examples, probably more for more complex programs.

Author: matthijskooijman

XBee.cpp

@@ -1014,53 +1014,6 @@ void PayloadRequest::setPayloadLength(uint8_t payloadLength) {
 	_payloadLength = payloadLength;
 }
 
-
-XBeeAddress::XBeeAddress() {
-
-}
-
-XBeeAddress64::XBeeAddress64() : XBeeAddress() {
-
-}
-
-XBeeAddress64::XBeeAddress64(uint32_t msb, uint32_t lsb) : XBeeAddress() {
-	_msb = msb;
-	_lsb = lsb;
-}
-
-XBeeAddress64::XBeeAddress64(uint64_t addr) : XBeeAddress() {
-	set(addr);
-}
-
-uint32_t XBeeAddress64::getMsb() {
-	return _msb;
-}
-
-void XBeeAddress64::setMsb(uint32_t msb) {
-	_msb = msb;
-}
-
-uint32_t XBeeAddress64::getLsb() {
-	return _lsb;
-}
-
-void XBeeAddress64::setLsb(uint32_t lsb) {
-	_lsb = lsb;
-}
-
-uint64_t XBeeAddress64::get() {
-	return (static_cast<uint64_t>(_msb) << 32) | _lsb;
-}
-
-void XBeeAddress64::set(uint64_t addr) {
-	_msb = addr >> 32;
-	_lsb = addr;
-}
-
-XBeeAddress64::operator uint64_t() {
-	return get();
-}
-
 #ifdef SERIES_2
 
 ZBTxRequest::ZBTxRequest() : PayloadRequest(ZB_TX_REQUEST, DEFAULT_FRAME_ID, NULL, 0) {

XBee.h

@@ -287,7 +287,7 @@ class XBeeResponse {
 
 class XBeeAddress {
 public:
-	XBeeAddress();
+	XBeeAddress() {};
 };
 
 /**
@@ -299,16 +299,19 @@ class XBeeAddress {
  */
 class XBeeAddress64 : public XBeeAddress {
 public:
-	XBeeAddress64(uint64_t addr);
-	XBeeAddress64(uint32_t msb, uint32_t lsb);
-	XBeeAddress64();
-	uint32_t getMsb();
-	uint32_t getLsb();
-	uint64_t get();
-	operator uint64_t();
-	void setMsb(uint32_t msb);
-	void setLsb(uint32_t lsb);
-	void set(uint64_t addr);
+	XBeeAddress64(uint64_t addr) : _msb(addr >> 32), _lsb(addr) {}
+	XBeeAddress64(uint32_t msb, uint32_t lsb) : _msb(msb), _lsb(lsb) {}
+	XBeeAddress64() {}
+	uint32_t getMsb() {return _msb;}
+	uint32_t getLsb() {return _lsb;}
+	uint64_t get() {return (static_cast<uint64_t>(_msb) << 32) | _lsb;}
+	operator uint64_t() {return get();}
+	void setMsb(uint32_t msb) {_msb = msb;}
+	void setLsb(uint32_t lsb) {_lsb = lsb;}
+	void set(uint64_t addr) {
+		_msb = addr >> 32;
+		_lsb = addr;
+	}
 private:
 	// Once https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66511 is
 	// fixed, it might make sense to merge these into a uint64_t.