[UI] Configurable Allocator for modm::ViewStack

src/modm/ui/display/color_graphic_display.hpp

@@ -5,8 +5,6 @@
 
 #include "graphic_display.hpp"
 
-using namespace modm::platform;
-
 namespace modm
 {
 

src/modm/ui/display/graphic_display.hpp

@@ -127,7 +127,7 @@ class GraphicDisplay : public IOStream
 	inline void
 	clearPixel(glcd::Point p)
 	{
-		this->setPixel(p.x, p.y);
+		this->clearPixel(p.x, p.y);
 	}
 
 	/**

src/modm/ui/gui/module.lb

@@ -23,9 +23,7 @@ def prepare(module, options):
         ":container",
         ":debug",
         ":processing:timer",
-        ":ui:display",
-        ":ui:gui",
-        ":ui:menu")
+        ":ui:display")
     return True
 
 def build(env):

src/modm/ui/gui/view.cpp

@@ -18,9 +18,10 @@
 
 // ----------------------------------------------------------------------------
 modm::gui::View::View(modm::gui::GuiViewStack* stack, uint8_t identifier, modm::gui::Dimension dimension) :
-	AbstractView(stack, identifier),
 	stack(stack),
-	dimension(dimension)
+	dimension(dimension),
+	identifier(identifier),
+	alive(true)
 {
 	this->display().clear();
 }
@@ -161,3 +162,9 @@ void modm::gui::View::markDrawn()
 		(*iter)->markDrawn();
 	}
 }
+
+modm::ColorGraphicDisplay&
+modm::gui::View::display()
+{
+	return stack->getDisplay();
+}

src/modm/ui/gui/view.hpp

@@ -24,8 +24,6 @@
 #include "widgets/widget.hpp"
 #include "colorpalette.hpp"
 
-#include <modm/ui/menu/abstract_view.hpp>
-
 namespace modm
 {
 
@@ -41,7 +39,7 @@ class GuiViewStack;
  * @ingroup modm_ui_gui
  * @author	Thorsten Lajewski
  */
-class View : public modm::AbstractView
+class View
 {
 	friend class GuiViewStack;
 
@@ -61,6 +59,14 @@ class View : public modm::AbstractView
 	virtual void
 	update();
 
+	/**
+	 * @brief hasChanged indicates the current displayed view has changed.
+	 *        This function prevents unnecessary drawing of the display
+	 * @return if true the display has to be redrawn.
+	 */
+	virtual bool
+	hasChanged() = 0;
+
 	virtual void
 	preUpdate()
 	{
@@ -79,10 +85,30 @@ class View : public modm::AbstractView
 	bool
 	pack(Widget *w, const modm::glcd::Point &coord);
 
+	/**
+	 * @brief isAlive tells the ViewStack if it should remove this screen.
+	 * @return
+	 */
+	bool
+	isAlive() const
+	{
+		return this->alive;
+	}
+
 	/// Remove the view from the screen. The viewStack handles the deletion.
 	void
 	remove();
 
+	/**
+	 * @brief onRemove will be called right before the view gets deleted,
+	 *        can be reimplemented to reset external data.
+	 */
+	virtual void
+	onRemove()
+	{
+		// nothing to be done here
+	}
+
 	/// Set color palette for every contained widget
 	void
 	setColorPalette(ColorPalette& cp);
@@ -105,12 +131,25 @@ class View : public modm::AbstractView
 		return stack;
 	}
 
+	modm::ColorGraphicDisplay&
+	display();
+
+	/**
+	 * @brief getIdentifier of the view.
+	 */
+	inline uint8_t getIdentifier(){
+		return this->identifier;
+	}
+
 protected:
 	modm::gui::GuiViewStack* stack;
 	Dimension dimension;
 	WidgetContainer widgets;
 
 	modm::gui::ColorPalette colorpalette;
+
+	const uint8_t identifier;
+	bool alive;
 };
 
 }	// namespace gui

src/modm/ui/gui/view_stack.cpp

@@ -20,7 +20,7 @@
 
 // ----------------------------------------------------------------------------
 modm::gui::GuiViewStack::GuiViewStack(modm::ColorGraphicDisplay* display, modm::gui::inputQueue* queue) :
-	ViewStack(display),
+	display(display),
 	input_queue(queue)
 {
 }

src/modm/ui/gui/view_stack.hpp

@@ -21,7 +21,6 @@
 #include <modm/ui/display/color_graphic_display.hpp>
 #include <modm/container/stack.hpp>
 #include <modm/container/linked_list.hpp>
-#include <modm/ui/menu/menu_buttons.hpp>
 #include "view.hpp"
 #include "types.hpp"
 
@@ -41,7 +40,7 @@ namespace gui
  * @ingroup modm_ui_gui
  * @author	Thorsten Lajewski
  */
-class GuiViewStack : public modm::ViewStack
+class GuiViewStack
 {
 public:
 	GuiViewStack(modm::ColorGraphicDisplay* display, modm::gui::inputQueue* queue);
@@ -84,7 +83,17 @@ class GuiViewStack : public modm::ViewStack
 	virtual void
 	update();
 
+	/**
+	 * @brief getDisplay access underlying GraphicDisplay
+	 */
+	inline modm::ColorGraphicDisplay&
+	getDisplay()
+	{
+		return *this->display;
+	}
+
 private:
+	modm::ColorGraphicDisplay *display;
 	modm::Stack< modm::gui::View* , modm::LinkedList< modm::gui::View* > > stack;
 	modm::gui::inputQueue *input_queue;
 };

src/modm/ui/menu/abstract_menu.hpp

@@ -30,12 +30,17 @@ namespace modm{
 	 * \author Thorsten Lajewski
 	 * \ingroup modm_ui_menu
 	 */
-	class AbstractMenu: public AbstractView
+	template<typename Allocator = std::allocator<IAbstractView> >
+	class AbstractMenu : public AbstractView<Allocator>
 	{
 		public:
 
-			AbstractMenu(modm::ViewStack* stack, uint8_t identifier);
+			AbstractMenu(modm::ViewStack<Allocator>* stack, uint8_t identifier) :
+				modm::AbstractView<Allocator>(stack, identifier)
+			{
+			}
 
+			virtual ~AbstractMenu() {}
 
 			virtual void
 			shortButtonPress(modm::MenuButtons::Button button) = 0;