prog_uint8_t

[Uwe (Gast)]

Hi,

kann ich sowas machen, oder handele ich mir damit Probleme ein?

typedef uint8_t prog_uint8_t;


prog_uint8_t can_filter[] =
{
  // Group 0
  MCP2515_FILTER(0),        // Filter 0
  MCP2515_FILTER(0),        // Filter 1
  
  // Group 1
  MCP2515_FILTER_EXTENDED(0),    // Filter 2
  MCP2515_FILTER_EXTENDED(0),    // Filter 3
  MCP2515_FILTER_EXTENDED(0),    // Filter 4
  MCP2515_FILTER_EXTENDED(0),    // Filter 5
  
  MCP2515_FILTER(0),        // Mask 0 (for group 0)
  MCP2515_FILTER_EXTENDED(0),    // Mask 1 (for group 1)
};

[Michael R. (Firma: Brainit GmbH) (fisa)]

Solange du uns nicht verrätst, was MCP2515_FILTER und 
MCP2515_FILTER_EXTENDED macht, wird dir das keiner beantworten können...

[Uwe (Gast)]

Michael Reinelt schrieb:
> Solange du uns nicht verrätst, was MCP2515_FILTER und
> MCP2515_FILTER_EXTENDED macht, wird dir das keiner beantworten können...

Mir geht um prog_uint8_t, was unter Atmel Studio nicht vorhanden ist. 
Der Rest ist erstmal irrelevant.

prog_uint8_t ist doch auch ein uin8_t, oder nicht?

[Grml .. (grml)]

Uwe schrieb:
> Michael Reinelt schrieb:
> Solange du uns nicht verrätst, was MCP2515_FILTER und
> MCP2515_FILTER_EXTENDED macht, wird dir das keiner beantworten können...
>
> Mir geht um prog_uint8_t, was unter Atmel Studio nicht vorhanden ist.
> Der Rest ist erstmal irrelevant.
>
> prog_uint8_t ist doch auch ein uin8_t, oder nicht?

Ja das ist ein Datentyp mit dem gleichen Wertebereich. Solltest du 
allerdings einer uint8_t Variable einen prog_uint8_t Wert zuweisen gibt 
es je nach Compilereinstellung mindestens mal ne Warnung.

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

Grml ... schrieb:
> Solltest du allerdings einer uint8_t Variable einen prog_uint8_t Wert
> zuweisen gibt es je nach Compilereinstellung mindestens mal ne Warnung.

Warum?

@TE: wofür brauchst du den typedef denn überhaupt?  Was mindestens
noch fehlt ist das "const" bei der Definition.

Aber sinnvoller wäre es, stattdessen dann gleich __flash zu benutzen,
wenn du sowieso einen hinreichend neuen Compiler benutzt.

[Michael R. (Firma: Brainit GmbH) (fisa)]

Uwe schrieb:
> prog_uint8_t ist doch auch ein uin8_t, oder nicht?

Ja, aber einer der im Flash liegt:

typedef uint8_t PROGMEM prog_uint8_t;

ist allerdings schon lange deprecated:

The typedef is only visible if the macro __PROG_TYPES_COMPAT__
   has been defined before including <avr/pgmspace.h> (either by a
   #define directive, or by a -D compiler option.)

[Grml .. (grml)]

Jörg Wunsch schrieb:
> Grml ... schrieb:
> Solltest du allerdings einer uint8_t Variable einen prog_uint8_t Wert
> zuweisen gibt es je nach Compilereinstellung mindestens mal ne Warnung.
>
> Warum?

Das hat etwas mit Typsicherheit zu tun, da der typdef nur eine Aussage 
über die Speichergröße und Wertebereich/Präzision gibt. Ein Beispiel ist 
boolean in c (auch eine Variable mit 8bit). Siehe auch Misra Rule 10.1.


Grml ... schrieb:
> ...gibt
> es je nach Compilereinstellung mindestens mal ne Warnung.

Falsch ausgedrückt. Muss heißen gibt bei entsprechender 
Compilereinstellung eine Warnung

[Uwe (Gast)]

Habe es nun folgendermaßen:

can_static_filter(const uint8_t PROGMEM *filter_array);

prog_uint8_t can_filter[] =
{
  // Group 0
  MCP2515_FILTER(0),        // Filter 0
  MCP2515_FILTER(0),        // Filter 1
  
  // Group 1
  MCP2515_FILTER_EXTENDED(0),    // Filter 2
  MCP2515_FILTER_EXTENDED(0),    // Filter 3
  MCP2515_FILTER_EXTENDED(0),    // Filter 4
  MCP2515_FILTER_EXTENDED(0),    // Filter 5
  
  MCP2515_FILTER(0),        // Mask 0 (for group 0)
  MCP2515_FILTER_EXTENDED(0),    // Mask 1 (for group 1)
};

Die typedef habe ich nun entfernt und fühle mich sicherer :-)

http://gcc.gnu.org/onlinedocs/gcc/Named-Address-Spaces.html#AVR%20Named%20Address%20Spaces

[Dr. Sommer (Gast)]

Grml ... schrieb:
> Das hat etwas mit Typsicherheit zu tun, da der typdef nur eine Aussage
> über die Speichergröße und Wertebereich/Präzision gibt.
Quatsch. Ein typedef ist einfach nur ein Typ-Alias. Deswegen ist 
folgender Code auch kein Problem und gibt keine Warnung:

typedef int foo;

int main () {
  foo x;
  x = 7;
  return 0;
}

Die 7 ist zwar vom Typ "int", aber x ist vom Typ "foo" - trotzdem keine 
Warnung, da "foo" und "int" das selbe ist.

Millionen Zeilen Code verlassen sich darauf dass typedef einfach nur ein 
Alias ist und man beliebig zuweisen kann...

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

Grml ... schrieb:
> Muss heißen gibt bei entsprechender Compilereinstellung eine Warnung

Nein, gibt es nicht, und darf es nicht geben.

prog_uint8_t war pure Augenauswischerei, und genau deshalb ist er auch
entsorgt worden.