fixing examples memory allocation on AVR's

some examples that do not use macros run well on non-avr devices but have mesleading memory allocation mode

Author: neu-rah

examples/LCDs/LiquidCrystal/LiquidCrystal/LiquidCrystal.ino

@@ -153,7 +153,7 @@ MENU(mainMenu,"Main menu",doNothing,noEvent,wrapStyle
 #define MAX_DEPTH 2
 /*idx_t tops[MAX_DEPTH];
 liquidCrystalOut outLCD(lcd,tops,pList);//output device for LCD
-menuOut* outputs[]={&outLCD};//list of output devices
+menuOut* constMEM outputs[] MEMMODE={&outLCD};//list of output devices
 outputsList out(outputs,1);//outputs list with 2 outputs*/
 
 MENU_OUTPUTS(out, MAX_DEPTH

examples/LCDs/Malpartida/I2C/I2C/I2C.ino

@@ -157,7 +157,7 @@ http://playground.arduino.cc/Code/LCD3wires
   panelsList pList(panels,nodes,1);
   idx_t tops[MAX_DEPTH];
   lcdOut outLCD(&lcd,tops,pList);//output device for LCD
-  menuOut* outputs[]={&outLCD};//list of output devices
+  menuOut* constMEM outputs[] MEMMODE={&outLCD};//list of output devices
   outputsList out(outputs,1);//outputs list with 2 outputs
   */
 

examples/SSD1306Ascii/SSD1306Ascii/SSD1306Ascii.ino

@@ -9,7 +9,7 @@
 
 #include <menu.h>
 #include <menuIO/SSD1306AsciiOut.h>
-#include <menuIO/serialIn.h>
+#include <menuIO/serialIO.h>
 using namespace Menu;
 
 SSD1306AsciiWire oled;
@@ -73,15 +73,15 @@ class altPrompt:public prompt {
 public:
   altPrompt(constMEM promptShadow& p):prompt(p) {}
   Used printTo(navRoot &root,bool sel,menuOut& out, idx_t idx,idx_t len,idx_t) override {
-    return out.printRaw(F("special prompt!"),len);
+    return out.printRaw(F( "special prompt!"),len);
   }
 };
 
 MENU(subMenu,"Sub-Menu",showEvent,anyEvent,noStyle
   ,OP("Sub1",showEvent,anyEvent)
   ,OP("Sub2",showEvent,anyEvent)
   ,OP("Sub3",showEvent,anyEvent)
-  ,altOP(altPrompt,"",showEvent,anyEvent)
+  // ,altOP(altPrompt,"",showEvent,anyEvent)
   ,EXIT("<Back")
 );
 
@@ -104,15 +104,19 @@ MENU(mainMenu,"Main menu",doNothing,noEvent,wrapStyle
 #define fontW 5
 #define fontH 8
 
+//define output device
+idx_t serialTops[MAX_DEPTH]={0};
+serialOut outSerial(Serial,serialTops);
+
 //describing a menu output device without macros
 //define at least one panel for menu output
 constMEM panel panels[] MEMMODE={{0,0,128/fontW,64/fontH}};
 navNode* nodes[sizeof(panels)/sizeof(panel)];//navNodes to store navigation status
 panelsList pList(panels,nodes,1);//a list of panels and nodes
 idx_t tops[MAX_DEPTH]={0,0};//store cursor positions for each level
 SSD1306AsciiOut outOLED(&oled,tops,pList);//oled output device menu driver
-menuOut* outputs[]={&outOLED};//list of output devices
-outputsList out(outputs,1);//outputs list
+menuOut* constMEM outputs[] MEMMODE ={&outSerial,&outOLED,};//list of output devices
+outputsList out(outputs,sizeof(outputs)/sizeof(menuOut*));//outputs list
 
 //macro to create navigation control root object (nav) using mainMenu
 serialIn serial(Serial);
@@ -154,8 +158,8 @@ void setup() {
   oled.begin(&Adafruit128x64, I2C_ADDRESS);
   oled.setFont(System5x7);
   oled.clear();
-  oled.print("Hello world!");
-  nav.idleTask=idle;//point a function to be used when menu is suspended
+  oled.print("menu 4.x test");
+  // nav.idleTask=idle;//point a function to be used when menu is suspended
 }
 
 void loop() {

examples/SSD1306Ascii/platformio.ini

@@ -16,9 +16,10 @@ platform = atmelavr
 board = nanoatmega328
 framework = arduino
 upload_port=/dev/ttyUSB*
-upload_flags=-V
+; upload_flags=-V
 upload_speed=57600
-build_flags = -DNODEBUG
+build_flags = -DMENU_DEBUG
+src_build_flags = !echo "-Wno-write-strings -Wno-reorder -DLOC="$PLATFORMIO_LOC
 
 ; [env:teensy31]
 ; platform = teensy

examples/Serial/ansiSerial/ansiSerial/ansiSerial.ino

@@ -54,7 +54,7 @@ const colorDef<uint8_t> colors[] MEMMODE={
 // panelsList pList(panels,nodes,sizeof(panels)/sizeof(panel));
 // idx_t ansi_tops[MAX_DEPTH];
 // ansiSerialOut ansi(Serial,colors,ansi_tops,pList);//the output device, ansi-terminal Cols x Rows
-// menuOut* outputs[]={&ansi};
+// menuOut* constMEM outputs[] MEMMODE={&ansi};
 // outputsList out(outputs,1);
 
 MENU_OUTPUTS(out,MAX_DEPTH

examples/clickEncoder/clickEncoder/clickEncoder.ino

@@ -143,12 +143,12 @@ MENU(mainMenu,"Main menu",doNothing,noEvent,wrapStyle
 
 #define MAX_DEPTH 2
 
-/*const panel panels[] MEMMODE={{0,0,16,2}};
+/*constMEM panel panels[] MEMMODE={{0,0,16,2}};
 navNode* nodes[sizeof(panels)/sizeof(panel)];
 panelsList pList(panels,nodes,1);
 idx_t tops[MAX_DEPTH];
 lcdOut outLCD(&lcd,tops,pList);//output device for LCD
-menuOut* outputs[]={&outLCD};//list of output devices
+menuOut* constMEM outputs[] MEMMODE={&outLCD};//list of output devices
 outputsList out(outputs,1);//outputs list with 2 outputs
 */
 

examples/dynamic/dynamic/dynamic.ino

@@ -131,7 +131,7 @@ idx_t tops[MAX_DEPTH];
 serialOut out(Serial,tops);
 
 //outputs -------------------------------------
-menuOut* outList[]={&out};
+menuOut* constMEM outList[] MEMMODE={&out};
 outputsList outs(outList,sizeof(outList)/sizeof(menuOut*));
 
 //navigation control -------------------------------------

src/items.cpp

@@ -38,7 +38,7 @@ Used prompt::printTo(navRoot &root,bool sel,menuOut& out, idx_t idx,idx_t len,id
     //   out.fmtStart(*this,menuOut::fmtBody,root.node(),idx);
     // #endif
     if (root.node().target==this) {
-      trace(MENU_DEBUG_OUT<<"printMenu"<<endl);
+      trace(MENU_DEBUG_OUT<<"calling printMenu"<<endl);
       out.printMenu(root.node(), panelNr);
     } else {
       trace(MENU_DEBUG_OUT<<"previewMenu"<<endl);

src/menuIo.cpp

@@ -137,7 +137,7 @@ idx_t gfxOut::editCursor(navRoot& root,idx_t x,idx_t y,bool editing,bool charEdi
 }
 
 Used outputsList::printMenu(navNode& nav) const {
-  trace(MENU_DEBUG_OUT<<"outputsList::printMenu"<<endl);
+  trace(MENU_DEBUG_OUT<<"outputsList::printMenu"<<endl;MENU_DEBUG_OUT.flush());
   for(int n=0;n<cnt;n++) {
     menuOut& o=*((menuOut*)memPtr(outs[n]));
     if (nav.changed(o)||(o.style&(menuOut::rasterDraw))||(o.style&(menuOut::redraw)))

src/nav.cpp

@@ -132,7 +132,7 @@ void navRoot::useMenu(menuNode &menu) {
 }
 
 Used navRoot::printMenu() const {
-  trace(MENU_DEBUG_OUT<<"navRoot::printMenu"<<endl);
+  trace(MENU_DEBUG_OUT<<"printMenu()"<<endl);
   if ((active().sysStyles()&_parentDraw)&&level)
     return out.printMenu(path[level-1]);
   else return out.printMenu(node());
@@ -155,6 +155,7 @@ void navRoot::doInput(menuIn& in) {
 }
 
 void navRoot::doOutput() {
+  trace(Serial<<"navRoot::doOutput "<<(int)sleepTask<<endl);
   if (!sleepTask) printMenu();
   else {
     bool c=idleChanged;