Add some thoughts in form of comments

src/Commands/CommandManager.h

@@ -1,6 +1,3 @@
-// CommandManager class to keep a stack of commands for undo functionality
-// Created by Krisjanis Rijnieks 2015-03-24
-
 #pragma once
 
 #import <deque>
@@ -14,9 +11,12 @@ namespace ofx{
             public:
                 void executeCommand(BaseCommand * cmd);
                 void undo();
+                // TODO: Add redo
             
             private:
                 std::deque<BaseUndoableCommand *> commandStack;
+                // TODO: Add redo stack
+                //       Maybe use vector instead of deque...
         };
         
     } // namespace piMapper

src/MediaServer/MediaServer.h

@@ -13,6 +13,10 @@
 
 #include "ofMain.h"
 #include "DirectoryWatcher.h"
+
+/* Discussion: This could be the right place for a Factory Method or
+ * Abstract Factory design pattern - replace all these includes with a
+ * SourceFactory that can create sources with the interfaces below. */
 #include "BaseSource.h"
 #include "ImageSource.h"
 #include "VideoSource.h"

src/ofxPiMapper.h

@@ -1,12 +1,5 @@
-/* ofxPiMapper
- *
- * Description: 
- * Main ofxPiMapper class. Holds instances of all necessary parts of the tool:
- *  - MediaServer
- *  - SurfaceManager
- *  - SurfaceManagerGui (this asks for a different architecture)
- *
- * Author: Krisjanis Rijnieks */
+// ofxPiMapper
+// Author: Krisjanis Rijnieks
 
 // On using prefixes like m or p (mMemberVariable, pMyPointer)
 // http://stackoverflow.com/questions/1228161/why-use-prefixes-on-member-variables-in-c-classes