GLCD Font Probleme (wahrscheinlich Pointer)

[N. G. (newgeneration)]

Hallo Forum,

ich schreibe gerade an einer Lib für ein GLCD (128x64).
Eigentlich funktioniert schon alles, aber:
ich will eine Funktion, die die aktuelle Schriftart auswählt, sodass ich 
insgesamt mehrere verschiedene Fonts haben kann.

Anbei mal der aktuelle Code (nicht aufgeräumt).
Ich vermute es liegt an der Funktion setFont, bzw. der Pointer-Operation 
darin.

Hier noch die mMn kritischen Stellen (für die die nicht alles anschauen 
wollen):

In fonts.h habe ich einen typedef für die verschiedenen Fonts und die 
Deklarationen von den Schriftarten:

typedef struct {
  const uint8_t width;
  const uint8_t height;
  const uint8_t text[]; //macht das das was ich will?
} font_t;

extern const __flash font_t dogm_font_6x8;
extern const __flash font_t dogm_font_4x6;

Die Font-Files sehen so aus:

const __flash font_t dogm_font_6x8 = {  //font 6x8 normal
    .width = 6,
    .height = 8,
    .text = { //font data
        0x00, 0x00, 0x00, 0x00, 0x00, 0x00, //die einzelnen Zeichen
        ...
        }};

und in der Haupt-Display-Datei:

static const font_t *currentFont;

void dogm_setFont(const font_t font) {
  currentFont = &font; //hier koennte das Problem sein
}

void dogm_setPos(uint8_t x, uint8_t y) {
  uint8_t i = 0;
  if (x > 127)
    dogm_setPos(0, y +          // hier die beiden moeglichkeiten: 
      currentFont->height);     //funktioniert nicht
      //dogm_font_6x8.height);  //funktioniert
  else {
    currentPosX = x;
    currentPosY = y + (i *      //hier die beiden moeglichkeiten: 
      currentFont->height);     //funktioniert nicht
      //dogm_font_6x8.height);  //funktioniert
  }
}

void dogm_putc(uint8_t c) {
  if (currentFont == 0) {
    return;
  }
  uint8_t width = currentFont->width;    //funktioniert nicht
  //uint8_t width = dogm_font_6x8.width; //funktioniert
  if (currentPosX + width > 127)
    dogm_setPos(currentPosX + width, currentPosY);
  for (uint8_t i = 0; i < width; i++)
    disp_ram[(currentPosY >> 3) + ((currentPosX + i) << 3)] =
      currentFont->text[c * width + i]; //funktioniert nicht
      //dogm_font_6x8.text[c*width+i];  //funktioniert
  dogm_setPos(currentPosX + width, currentPosY);
}

Also es wäre schön wenn jemand mal einen Blick riskieren könnte und 
schauen ob es überhaupt so geht wie ich will.

Ohne diesen Schriftartenwechsel geht alles einwandfrei (okay, man kann 
über den Shadow-RAM streiten, aber aktuell geht es ;) )

Danke im Vorraus
N.G.

[Stefan E. (sternst)]

void dogm_setFont(const font_t font) {
  currentFont = &font; //hier koennte das Problem sein
}

Einen globalen Pointer auf eine funktions-lokale Variable, die nach dem 
Ende der Funktion gar nicht mehr existiert, zeigen zu lassen, ist in der 
Tat eine ziemlich schlechte Idee.

[N. G. (newgeneration)]

Hallo Stefan,

Stefan E. schrieb:
> Einen globalen Pointer auf eine funktions-lokale Variable, die nach
> dem Ende der Funktion gar nicht mehr existiert, zeigen zu lassen, ist in
> der Tat eine ziemlich schlechte Idee.

Du meinst die Zuweisung ist schon richtig, aber bei Verlassen der 
Funktion wird die lokale Variable zerstört und der Pointer zeigt auf nix 
sinnvolles mehr?
Habe davon noch nie etwas gehört, aber ist ja irgendwie auch logisch...
So nun zum Quick-Fix:
Wie umgehen ich das? Mit einem Pointer als Parameter? Oder ganz anders?

Danke schon mal

[Karl H. (kbuchegg)]

Da gibts noch ein paar mehr 'Probleme'.

Was willst du denn effektiv erreichen? Im Endeffekt soll doch alles 
darauf hinauslaufen, dass es zb auf ein

  currentFont = &dogm_font_6x8;

hinausläuft.
Denn dann ist currentFont ja als Zeiger so eingerichtet, dass er die 
Adresse des Fonts enthält derr benutzt werden soll.

Dazu müssen aber die Datentypen stimmen.
dogm_font_6x8 hat welchen Datentyp?

...   const __flash font_t dogm_font_6x8;
      ********************

Und worauf kann currentFont zeigen?

  ... const font_t *currentFont;

      ************

tja. Fast. dogm_font_6x8  hat noch die Attributierung '__flash' und es 
ist auch wichtig, dass die nicht verloren geht!
D.h. aber auch: dein Pointer muss auch in der Lage sein, auf etwas zu 
zeigen, was im Flash residiert.

  ... const __flash font_t *currentFont;

so, jetzt tut er das.

Wenn das Ziel darin besteht, dass letzten Endes der Effekt von

  currentFont = &dogm_font_6x8;

sich ergeben soll UND du das ganze über eine Funktion abwickeln willst, 
dann wirst du wohl nicht umhin kommen, dass der ganze rechte Teil der 
Zuweisung als Funktionsargument an eine Funktion übergeben wird.

D.h. du willst deinen Aufruf so gestalten

...
   dogm_setFont( &dogm_font_6x8 );
...

damit das möglich ist, muss daher die Funktions Argument Schnittstelle 
wie aussehen?

[N. G. (newgeneration)]

D.h. ich muss die Methode so abändern?

static const __flash font_t *currentFont;

void dogm_setFont(const __flash font_t *font) {
  currentFont = font; // kann ich das dann einfach so zuweisen?
}

Danke schon mal für die ausführliche Erklärung

[N. G. (newgeneration)]

Okay, danke an alle, mit den obigen Änderungen geht es jetzt thumbs up

[Eric B. (beric)]

N. G. schrieb:

> typedef struct {
>   const uint8_t width;
>   const uint8_t height;
>   const uint8_t text[]; //macht das das was ich will?
> } font_t;
> 
> extern const __flash font_t dogm_font_6x8;
> extern const __flash font_t dogm_font_4x6;

N. G. schrieb:
> Okay, danke an alle, mit den obigen Änderungen geht es jetzt thumbs up

Sicher?  Beschwert sich der Compiler nicht an der Stelle wo du schreibst 
"macht das das was ich will?" ??

[Karl H. (kbuchegg)]

https://en.wikipedia.org/wiki/Flexible_array_member

[W.S. (Gast)]

N. G. schrieb:
> ich schreibe gerade an einer Lib für ein GLCD (128x64).
> Eigentlich funktioniert schon alles, aber:
> ich will eine Funktion, die die aktuelle Schriftart auswählt, sodass ich
> insgesamt mehrere verschiedene Fonts haben kann.

Soweit ich das in deinen geposteten Dateien sehen kann, bist du 
ausgesprochen wirr an die ganze Sache herangegangen. Ich rate dir 
deshalb dringendst, das Ganze in die Tonne zu hauen und nochmal ganz von 
vorn anzufangen.

Du brauchst etwas, das man ein grafisches Frontend oder GDI nennen 
könnte, was all die Funktionen enthält, die man eben so zum Benutzen 
eines GLCD's braucht. Also Punkte setzen, Linien zeichnen, Blöcke 
einfärben, Text an beliebigen Positionen zeichnen und so. OK, du 
brauchst dazu auch Fonts.

Mein Rat: guck dir gdi.c, gdi.h und die fonts in der Lernbetty hier im 
Forum an, da ist bereits fast alles enthalten, was du brauchst. All 
dieses arbeitet auf einem RAM-internen Bildspeicher, der nach 
"Beschmutzung" sprich Modifilation anschließend ins Display geschaufelt 
wird. Ich glaub, ich hatte das sogar selektiv getan aus 
Geschwindigkeitsgründen.

Einen kleinen Fontcompiler hat's dazu auch, wo du dir quasi 
symbolgrafisch einen Font nach deinem Gusto kreieren kannst. Selbst ein 
kleiner Bilder-Compiler ist dabei (Bild-->Cquelle).

Nochwas zu Fonts: Es verstehen viele Leute den Umstand nicht, daß man 
beim Anwenden von Fonts Ascent und Descent beachten sollte, also die 
Quasi-Schreiblinie in einer Zeile gleichhalten sollte - sonst sieht's 
doof aus. Dazu braucht es aber die entsprechenden Angaben im Font - und 
sowas vermisse ich bei deinen Dateien komplett.

W.S.