Adding an optional loop function to the server call

src/PacketEvent.cpp

@@ -1,8 +1,11 @@
 #include "PacketEvent.h"
 
-PacketEventAbstract::PacketEventAbstract(uint32_t id) {
+PacketEvent::PacketEvent(uint32_t id) {
 	myId = id;
 }
-uint32_t PacketEventAbstract::getId() {
+PacketEventAbstract::PacketEventAbstract(uint32_t id) :
+		PacketEvent(id) {
+}
+uint32_t PacketEvent::getId() {
 	return myId;
 }

src/PacketEvent.h

@@ -1,15 +1,16 @@
-#ifndef  PacketEvent
-#define PacketEvent
+#ifndef  PacketEvent_HEADER
+#define PacketEvent_HEADER
 #include <stdint.h>
-class PacketEventAbstract {
+
+class PacketEvent {
 protected:
 	// Unique id of the function being called
 	uint32_t myId;
 public:
 	bool noResponse = false;
 	// Packet ID needs to be set
-	PacketEventAbstract(uint32_t id);
-	virtual ~PacketEventAbstract() {}
+	PacketEvent(uint32_t id);
+	virtual ~PacketEvent(){}
 	//User function to be called when a packet comes in
 	// Buffer contains data from the packet coming in at the start of the function
 	// User data is written into the buffer to send it back 
@@ -18,9 +19,22 @@ class PacketEventAbstract {
 	 * An optional loop function
 	 * If your server needs a pulse to keep it updated, implement this function
 	 */
-	virtual bool loop(){return false;}
+	virtual bool loop()=0;
 
 	uint32_t getId();
 };
 
+class PacketEventAbstract : public PacketEvent{
+public:
+	// Packet ID needs to be set
+	PacketEventAbstract(uint32_t id);
+	virtual ~PacketEventAbstract() {}
+	/**
+	 * An optional loop function
+	 * If your server needs a pulse to keep it updated, implement this function
+	 */
+	virtual bool loop(){return false;};
+};
+
+
 #endif /* end of include guard: PacketEvent */

src/SimplePacketComs.cpp

@@ -17,12 +17,12 @@ uint32_t SimplePacketComsAbstract::getNumberOfFloatsInPacket() {
 }
 
 void SimplePacketComsAbstract::attach(
-		PacketEventAbstract * eventImplementation) {
+		PacketEvent * eventImplementation) {
 	fmap[eventImplementation->getId()] = eventImplementation;
 }
 
-PacketEventAbstract * SimplePacketComsAbstract::detach(uint32_t id) {
-	PacketEventAbstract *event = fmap[id];
+PacketEvent * SimplePacketComsAbstract::detach(uint32_t id) {
+	PacketEvent *event = fmap[id];
 	fmap.erase(id);
 	return event;
 }
@@ -31,14 +31,17 @@ PacketEventAbstract * SimplePacketComsAbstract::detach(uint32_t id) {
  * This runs the packet server and calls all events if a backet comes in
  */
 void SimplePacketComsAbstract::server() {
+	for (std::map<uint32_t, PacketEvent*>::iterator it = fmap.begin();
+			it != fmap.end(); ++it) {
+		it->second->loop();
+	}
 	if (isPacketAvailible()) {
 		getPacket(buffer, numberOfBytes);
 		uint32_t currentId = getIdPointer()[0];
-		for (std::map<uint32_t, PacketEventAbstract*>::iterator it =
+		for (std::map<uint32_t, PacketEvent*>::iterator it =
 				fmap.begin(); it != fmap.end(); ++it) {
 			if (it->second->getId() == currentId) {
 				it->second->noResponse = false; // reset the response flag
-				it->second->loop();
 				it->second->event(getDataPointer());
 
 				if (it->second->noResponse == false) {

src/SimplePacketComs.h

@@ -16,7 +16,7 @@ class SimplePacketComsAbstract {
 
 	uint32_t numberOfBytes;
 	uint8_t * buffer;
-	std::map<uint32_t, PacketEventAbstract*> fmap;
+	std::map<uint32_t, PacketEvent*> fmap;
 public:
 	SimplePacketComsAbstract();
 	/**
@@ -49,14 +49,14 @@ class SimplePacketComsAbstract {
 	/**
 	 * Attach a function to a packet event
 	 */
-	void attach(PacketEventAbstract * eventImplementation);
+	void attach(PacketEvent * eventImplementation);
 	/**
 	 * Detach a function from a packet event.
 	 *
 	 * @param id The packet id.
 	 * @return The function that was attached.
 	 */
-	PacketEventAbstract * detach(uint32_t id);
+	PacketEvent * detach(uint32_t id);
 	/**
 	 * This runs the packet server and calls all events if a backet comes in
 	 */
@@ -72,7 +72,7 @@ class SimplePacketComsAbstract {
 	float * getDataPointer() {
 		return (float *) (buffer + 4);
 	}
-	std::map<uint32_t, PacketEventAbstract*> * getfMap() {
+	std::map<uint32_t, PacketEvent*> * getfMap() {
 		return &fmap;
 	}