Dummy für __flash bzw PROGMEN

[D a v i d K. (oekel)]

Hi,

ich würde gerne eine große Menge Strings, die ich für den  avr-gcc 
Compiler definiert habe im normalen gcc verwenden.

Daher bin ich auf der Suche nach einem geschickten Makro, dass mir diese 
attribute wieder entfernt/redefiniert.

...
#ifdef GCC
    #define __flash ?
    #define PROGMEM ??
#endif

const __flash cahr strHund[] = "Hund";
const char strKatze[] PROGMEM = "Katze";
...

Mir ist klar, dass ich alles doppelt definieren könnte a la:

...
#ifdef GCC
    const _cahr strHund[] = "Hund";
    const char strKatze[] = "Katze";
#else
const __flash cahr strHund[] = "Hund";
const char strKatze[] PROGMEM = "Katze";
#endif
...

Diese Pflege möchte ich mir jedoch sparen, auch wenn ein CnP mit 
modernen IDEs, die "Column Selection Mode" unterstützen recht schnell 
geht.

Grüße David

[Walter T. (nicolas)]

Statt der <pgmspace.h> "include" ich in meinen gemischen Projekten die 
folgendete "pgmspace.h":

#ifndef PGMSPACE_H
    #define PGMSPACE_H

    #ifdef AVR
      #define constflash const __flash
    #include <avr/pgmspace.h>
        #include <stdio.h>

  #elif defined(STM32F10x) || defined(STM32F4XX) || defined __MINGW64__ \
        || defined __MING32__ || defined __linux__
        #define PGMSPACE_EMULATION 1

    #else
        #error "undefined platform"

    #endif


    #if PGMSPACE_EMULATION
    #define PROGMEM
    #define pgm_read_byte(a) (*(a))
    #define pgm_read_word(a) (*(a))
    #define pgm_read_pointer(a) (*(a))
    #define constflash const
    #define PSTR(a) (a)

    #define strlen_P strlen
    #define strncpy_P strncpy

    #define vsnprintf_P vsnprintf
    #define printf_P printf
    #define fprintf_P fprintf
    #define fputs_P fputs

        // snprintf_P sollte wegen eines Fehlers in einer AVR-GCC-Version nicht verwendet
        // werden. Als Workaround wird die Funktion wasnprintf_P eingefuehrt.
    #define snprintf_P 1 // Fuehrt zu einem Fehler bei einem Build mit der Funktion
                         // snprintf_P()

    // Workaround-snprintf-Funktion
    #define wasnprintf_P snprintf

  #endif
#endif // PGMSPACE_H

(Vor dem Edit stand hier nur eine "Prinzip-Variante")

[Rufus Τ. F. (rufus)]

D a v i d K. schrieb:
> #ifdef GCC
>     #define __flash ?
>     #define PROGMEM ??
> #endif

Hier musst Du nur die Fragezeichen entfernen, dann wird das #define 
durch einen leeren String ersetzt.

Allerdings: Ist "GCC" bei Verwendung des AVR-GCC ganz sicher nicht 
definiert?

[Harry L. (mysth)]

Schau mal hier ab Zeile 162:
https://github.com/HarryLipphaus/DisplayKit/blob/master/Src/lcd.h

[Oliver S. (oliverso)]

Rufus Τ. F. schrieb:
> Allerdings: Ist "GCC" bei Verwendung des AVR-GCC ganz sicher nicht
> definiert?

avr-gcc ist ein gcc. Also ist das auch definiert.

Oliver

[Niklas G. (erlkoenig)]

Oliver S. schrieb:
> avr-gcc ist ein gcc. Also ist das auch definiert.

Der GCC definiert "__GNUC__" aber nicht "GCC". Da es sich um ein 
Compiler-Spezifisches Makro handelt, darf es nicht mit eventuellen 
Bezeichnern im User-Code kollidieren. Stell dir vor jemand hat dieses 
absolut korrekte und standardkonforme C-Programm verfasst:

int main () {
  int GCC = 42;
  return 0;
}

Wenn der GCC ein Makro namens "GCC" definieren würde, würde dieser 
standarkonforme Code nicht mehr funktionieren, wodurch der GCC nicht 
standardkonform wäre. Daher fängt das Makro mit 2 Unterstrichen an. 
Würde jemand ein Programm dieser Art verfassen:

int main () {
  int __GNUC__ = 42;
  return 0;
}

funktioniert es vielleicht in manchen Compilern, aber da Bezeichner mit 
2 Unterstrichen im User-Code verboten sind, ist es falsch und nicht 
standard-konform. Daher ist es ok wenn dieses Programm nicht im GCC 
funktioniert, was auch der Fall ist, da der GCC "__GNUC__" definiert.

Also: Kein Compiler wird ein Makro namens "GCC" definieren oder sonst 
irgendein compilerspezifisches Makro ohne 2 Unterstriche oder 
Unterstrich+Großbuchstabe. "__GNUC__" ist das was man will.

Siehe auch hier, es gibt noch ein paar mehr Details zu beachten:
https://sourceforge.net/p/predef/wiki/Compilers/

[Jörg W. (dl8dtl) (Moderator)]

PROGMEM kannst du gern in eigenem Code definieren, aber __flash gehört 
zum implementation namespace, davon solltest du die Finger lassen.

Umschreib' das besser komplett anders, bspw.:

#ifdef __AVR__
#  define FLASHSTRING(sym, str) const char sym[] __flash = str
#else
#  define FLASHSTRING(sym, str) const char sym[] = str
#endif

// ...
FLASHSTRING(msg1, "Hello, world!\n");
FLASHSTRING(msg2, "Good bye!\n");

[Rufus Τ. F. (rufus)]

Oliver S. schrieb:
> avr-gcc ist ein gcc. Also ist das auch definiert.

Und damit ist es hier unbrauchbar, um zwischen GCC für AVR und GCC für 
das Wirtssystem zu unterscheiden. Deshalb fragte ich danach.

Also ist

#ifndef __AVR__
    #define __flash
    #define PROGMEM
#endif

eine simple Lösung.

Jörgs Ansatz ist natürlich der bessere, erfordert aber mehr Arbeit.