String aus Progmem lesen

[Jürgen (Gast)]

Hallo ihr C-Gurus,

ich hätte da mal ein Problem im Atmel Studio 6.

Ich möchte einen mittels PSTR im Flash abgelegten String in eine 
Variable lesen um diesen zu bearbeiten und später anzuzeigen. Ich habe 
zur Dokumentation mal folgenden (nicht zwingend sinnigen) Testcode 
geschrieben:

int8_t msg_str[10];

void print_msg(char *t) {
  for (uint8_t index = 0; ; index++) {
    if (pgm_read_byte(&t[index]) == 0x0) {
      break;
    } else {
      msg_str[index] = pgm_read_byte(&t[index]);
    }
  }

// weitere Verarbeitung von msg_str

}

void print_str_from_progmem(void) {
  print_msg(PSTR("Test"));
}

Beim Kompilieren erhalte ich dann in der Zeile print_msg(PSTR("Test")); 
folgende Fehler:

Error  1  invalid conversion from 'const char*' to 'char*' 
[-fpermissive]
Error  2  initializing argument 1 of 'void print_msg(char*)' 
[-fpermissive]

Kann mir jemand auf die Sprünge helfen?

Gruß
Jürgen

[Karl H. (kbuchegg)]

Jürgen schrieb:

> void print_msg(char *t) {

void print_msg( const char * t)


gewöhn dir const korrektes Arbeiten an. Unabhängig davon, ob der 
COmpiler bei PROGMEM Sachen das const sehen will oder nicht. Will eine 
Funktion an ihren Aufrufer die Zusicherung geben "Ich werde über den 
Pointer den du mir gibst, NICHT versuchen die dahinter liegenden Daten 
zu verändern", dann drückt sie das auch in der Parameterliste mit einem 
"const" aus.

[Karl H. (kbuchegg)]

Wieso

  for (uint8_t index = 0; ; index++) {
    if (pgm_read_byte(&t[index]) == 0x0) {
      break;
    } else {
      msg_str[index] = pgm_read_byte(&t[index]);
    }

liest du den Character 2 mal aus dem Flash?
Bruacht doch keiner

  index = 0;
  while( ( c = pgm_read_byte(&t[index]) ) )
  {
    // mach was mit c, zb in msg_str speichern
    msg_str[index++] = c;
  }
  msg_str[index] = '\0';
  ...

[Jürgen (Gast)]

Hallo Karl Heinz,

danke für deine Tipps, werde mich bessern ;-)

Gruß
Jürgen

[Frank M. (ukw) (Moderator)]

Da hat Dir Karl Heinz direkt noch einen 2. Fehler stillschweigend mit 
ausgebügelt:

msg_str[index] = '\0';

Die String-Terminierung hattest du nämlich auch vergessen.