deactivated non pgm modes due to bugs...

Author: neu-rah

README.md

@@ -51,17 +51,6 @@ Generic keyboard (no PCINT) - configurable for digital or analog keyboards
 
 ClickEncoder https://github.com/0xPIT/encoder
 
-## Saving memory
-Some menu data can be stored on avr programming memory (flash) to save ram for your project.
-
-to activate memory saving put
-```c++
-#define USEPGM
-```
-on top of your code before any menu includes.
-
-this is conditional because some non-avr devices might not supoprt it.
-
 ## Menu definition example
 example of menu definition (c++ macros)
 

src/macros.h

@@ -1,15 +1,16 @@
-
 #ifdef USEPGM
   //storing some values into avr flash memory (saving ram space)
   #include <avr/pgmspace.h>
   #define MEMMODE PROGMEM
   #define pgmPtrNear(addr) pgm_read_ptr_near(addr)
-  #define pgmByteNear(addr) pgm_read_byte_near(addr)
+  #define pgmByteNear(addr) (pgm_read_byte_near(addr))
+  //#error USEPGM (just checking)
 #else
   //use ram for non-avr devices
   #define MEMMODE
   #define pgmPtrNear(addr) (addr)
-  #define pgmByteNear(addr) (*((byte*)addr))
+  #define pgmByteNear(addr) (*(uint8_t*)addr)
+  #error NO USEPGM (this mem mode is under development)
 #endif
 
 class prompt;

src/menu.cpp

@@ -27,6 +27,7 @@ prompt menu::exitOption(menu::exit);
 bool menu::wrapMenus=false;
 menuNode* menuNode::activeNode=NULL;
 
+//PROGMEM AUX PRINT
 void print_P(menuOut& s,const char* at) {
   int len=strlen_P(at);
   for(;len;len--,at++)

src/menu.h

@@ -25,6 +25,9 @@ v2.1 - Add full support of SetPosition(x,y) to move the menu inside the screen (
 #ifndef RSITE_ARDUINO_MENU_SYSTEM
   #define RSITE_ARDUINO_MENU_SYSTEM
 
+  //PGM activated by default, non pgm mode is not working ok
+  #define USEPGM
+
 	#include <Stream.h>
 	#include <HardwareSerial.h>
 
@@ -188,4 +191,6 @@ v2.1 - Add full support of SetPosition(x,y) to move the menu inside the screen (
     virtual bool isMenu() const {return true;}
   };
 
+  //PROGMEM AUX PRINT
+  void print_P(menuOut& s,const char* at);
 #endif