passing const argument Konstante an funktion übergeben

[Holger K. (holgerkraehe)]

Hallo zusammen

Ich habe folgenden Code:

zwei strukte.
Wobei das untere das obere enthält.

Nun frage ich mich grundsätzlich: werden hier unnötige Daten kopiert?
Denn nach einem Pointer sieht mir das ja nicht aus.

// Character descriptors for Arial 36pt
// { [Char width in bits], [Offset into arial_36ptCharBitmaps in bytes] }
const FONT_CHAR_INFO arial_36ptDescriptors[] =
{
  {23, 0},    // 0
  {13, 115},    // 1
  {23, 180},    // 2
  {23, 295},    // 3
  {24, 410},    // 4
  {23, 530},    // 5
  {24, 645},    // 6
  {23, 765},    // 7
  {23, 880},    // 8
  {23, 995},    // 9
  {4, 1110},    // :
};

// Font information for Arial 36pt
const FONT_INFO arial_36ptFontInfo =
{
  35, //  Character height
  '0', //  Start character
  ':', //  End character
  2, //  Width, in pixels, of space character
  arial_36ptDescriptors, //  Character descriptor array
  arial_36ptBitmaps, //  Character bitmap array
};

Leider bekomme ich dazu noch eine Warnung.

warning: initialization discards 'const' qualifier from pointer target type [enabled by default]

Könnte mich hier jemand etwas über die Hintergründe aufklären?
Dass arial.... nicht als const im unteren strukt deklariert ist, ist mir 
klar. aber einfach "const" davor bringt nichts.

Danke schonmal

[Tom (Gast)]

Was steckt hinter FONT_CHAR_INFO und FONT_INFO ?

[Holger K. (holgerkraehe)]

Stimmt, das müsste ich auch noch posten:

////////////////////////////////////////////////////////////////////////////////
typedef struct _font_char_info
{
   uint8_t Charwidth;
   uint16_t Offset;
}FONT_CHAR_INFO;
////////////////////////////////////////////////////////////////////////////////
typedef struct _font_info
{
   uint8_t CharacterHeight;
   char StartCharacter;
   char EndCharacter;
   uint8_t  FontSpace;
   FONT_CHAR_INFO *Descriptors;
   uint8_t *Bitmaps;
}FONT_INFO;

[Tom (Gast)]

FONT_CHAR_INFO *Descriptors ist ein normaler Pointer, also ist das, 
worauf er zeigt, problemlos manipulierbar, wenn man ein FONT_INFO hat.

Du versuchst, die Adresse eines als const definierten Arrays darin zu 
speichern, die const-ness ginge damit verloren.


Wenn man über das FONT_INFO das FONT_CHAR_INFO ändern können soll, darf 
arial_36ptDescriptors[] nicht const sein.
Wenn man das FONT_CHAR_INFO nicht ändern können muss, sollte Descriptors 
const sein.
Wenn die Header nicht änderbarer Code einer altlastenbehafteten 
Fremd-Library sind, hat man verloren und muss arial_36ptDescriptors 
nicht const machen, casten oder Warnungen abstellen, const-correctness 
geht nur ganz oder gar nicht.

[Harry L. (mysth)]

const bedeutet für den Compiler sinngemäß: "don't touch"
Wenn du einem Pointer die Adresse einer const-Variable zuweist, kann der 
Compiler nicht mehr sicherstellen, daß die Variable auch beim Zugriff 
über den Pointer konstant bleibt, und reagiert mit dieser Warnung.

Alles nix Schlimmes, so lange man weis, was man tut ;)

[Holger K. (holgerkraehe)]

Tom schrieb:
> FONT_CHAR_INFO *Descriptors ist ein normaler Pointer, also ist
> das,
> worauf er zeigt, problemlos manipulierbar, wenn man ein FONT_INFO hat.
>
> Du versuchst, die Adresse eines als const definierten Arrays darin zu
> speichern, die const-ness ginge damit verloren.
>
> Wenn man über das FONT_INFO das FONT_CHAR_INFO ändern können soll, darf
> arial_36ptDescriptors[] nicht const sein.
> Wenn man das FONT_CHAR_INFO nicht ändern können muss, sollte Descriptors
> const sein.
> Wenn die Header nicht änderbarer Code einer altlastenbehafteten
> Fremd-Library sind, hat man verloren und muss arial_36ptDescriptors
> nicht const machen, casten oder Warnungen abstellen, const-correctness
> geht nur ganz oder gar nicht.

Danke für deine Antwort.
Nein es ist nicht altlastenbehaftet. Sondern es wird neu programmiert.
Die Fonts kommen aus diesem tool: 
http://www.eran.io/the-dot-factory-an-lcd-font-and-image-generator/

Es muss nichts änderbar sein an den Structs.
Sie dienen lediglich dem auslesen und sollen möglichst im Flash landen.


Wo soll ich Descriptors als const definieren?

So?

////////////////////////////////////////////////////////////////////////////////
typedef const struct _font_char_info
{
   uint8_t Charwidth;
   uint16_t Offset;
}FONT_CHAR_INFO;
////////////////////////////////////////////////////////////////////////////////
typedef const struct _font_info
{
   uint8_t CharacterHeight;
   char StartCharacter;
   char EndCharacter;
   uint8_t  FontSpace;
   FONT_CHAR_INFO *Descriptors;
   uint8_t *Bitmaps;
}FONT_INFO;

[Holger K. (holgerkraehe)]

Harry L. schrieb:
> kann der
> Compiler nicht mehr sicherstellen, daß die Variable auch beim Zugriff
> über den Pointer konstant bleibt, und reagiert mit dieser Warnung.
>
> Alles nix Schlimmes, so lange man weis, was man tut ;)

Grundsätzlich weiss ich was ich tue :)
Ich schreibe nie in den Font.

Allerdings möchte ich, dass der Font durch das "const" im Flash landet.
Deshalb hoffe ich de compiler macht das auch noch, auch wenn er denkt 
ich könnte ja plötzlich was ändern aufgrund des pointers...

[Harry L. (mysth)]

Holger K. schrieb:
> Allerdings möchte ich, dass der Font durch das "const" im Flash landet.

Das erreichst du aber nicht durch const.

Wenn es sich um einen AVR handelt brauchst du PROGMEN.

[Holger K. (holgerkraehe)]

Harry L. schrieb:
> Holger K. schrieb:
>> Allerdings möchte ich, dass der Font durch das "const" im Flash landet.
>
> Das erreichst du aber nicht durch const.
>
> Wenn es sich um einen AVR handelt brauchst du PROGMEN.

Nein, es handelt sich um einen STM32 CM3 mit ARM-GCC Compiler

[Harry L. (mysth)]

Holger K. schrieb:
> Nein, es handelt sich um einen STM32 CM3 mit ARM-GCC Compiler

Dann ist doch alles gut.
Konstanten im Source landen auch ohne spezielle Deklaration im Flash.

[Holger K. (holgerkraehe)]

Gut, und wie sieht es mit dem eigentlichen problem aus?

Kann ich die Warnung irgendwie wegbringen indem ich ein strukt als const 
deklariere?

[(prx) A. K. (prx)]

Holger K. schrieb:
> Kann ich die Warnung irgendwie wegbringen indem ich ein strukt als const
> deklariere?

  const FONT_CHAR_INFO *Descriptors;