Leere #define

[ritzeratze (Gast)]

Hallo und guten Morgen,
kann mir bitte jemand den Kopf von der Angehängten Funktion erklären.
Was bedeutet die leere define mit idata?
Was macht das für einen Sinn?

Danke
Schönes Wochenende!

#define uchar unsigned char
#define uint unsigned int
#define bit uchar
#define idata
#define code


uchar w1_rom_search( uchar diff, uchar idata *id )
{
  uchar i, j, next_diff;
  bit b;

  if( w1_reset() )
    return PRESENCE_ERR;      // error, no device found
  w1_byte_wr( SEARCH_ROM );      // ROM search command
  next_diff = LAST_DEVICE;      // unchanged on last device
  i = 8 * 8;          // 8 bytes
  do{
    j = 8;          // 8 bits
    do{
      b = w1_bit_io( 1 );      // read bit
      if( w1_bit_io( 1 ) ){      // read complement bit
  if( b )          // 11
    return DATA_ERR;      // data error
      }else{
  if( !b ){        // 00 = 2 devices
    if( diff > i ||
      ((*id & 1) && diff != i) ){
      b = 1;        // now 1
      next_diff = i;      // next pass 0
    }
  }
      }
      w1_bit_io( b );           // write bit
      *id >>= 1;
      if( b )          // store bit
  *id |= 0x80;
      i--;
    }while( --j );
    id++;          // next byte
  }while( i );
  return next_diff;        // to continue search
}

[PittyJ (Gast)]

idata wird durch 'nichts' ersetzt. Der Kopf wird also zu

uchar w1_rom_search( uchar diff, uchar  *id )

und durch die uchar ersetzung zu

unsigned char w1_rom_search( unsigned char diff, unsigned char  *id )


Schlechter Stil ist das allemal. Defines schreibt man meistens groß, 
also
#define UCHAR unsigned char

Und die meisten defines sind überflüssig. Besser typedef benutzen.

[Oliver S. (oliverso)]

Schreib dir halt einfach mal auf, was die Textersetzung aus dem Code 
macht.

Vermutlich ist das Code, der für Unterschiedliche Plattformen kompiliert 
werden kann, und manche davon benötigen halt noch spezielle qualifier.

Oliver

[ritzeratze (Gast)]

Ach jetzt Blick ich durch!
DAnke

[Peter D. (peda)]

Oliver S. schrieb:
> Vermutlich ist das Code, der für Unterschiedliche Plattformen kompiliert
> werden kann, und manche davon benötigen halt noch spezielle qualifier.

Genau so ist es.
Der Code ist original für den Keil C51 und der kennt z.B. auch 
Bitvariablen.
Ich hab viel Code auf 8051 und auf AVR laufen und da wäre es dumm, den 
gesamten Code umzuschreiben.

[Masl (Gast)]

Keine Ahnung wieso, aber als ich den Code sah dachte ich mir spontan: 
"Der kommt bestimmt von PeDa" ;-)

[Mark B. (markbrandis)]

Peter Dannegger schrieb:
> Der Code ist original für den Keil C51 und der kennt z.B. auch
> Bitvariablen.
> Ich hab viel Code auf 8051 und auf AVR laufen

Gibt's da kein stdint.h?

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

Mark Brandis schrieb:
> Gibt's da kein stdint.h?

Was hülfe dir das bezüglich „bit“, „idata“ und „code“?

[Mark B. (markbrandis)]

Jörg Wunsch schrieb:
> Was hülfe dir das bezüglich „bit“, „idata“ und „code“?

Es würde bezüglich uint und uchar helfen, und indirekt dann auch bei bit 
weil dieses ja auch über uchar definiert ist.

Drei von fünf ist doch nicht schlecht :)

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

Mark Brandis schrieb:
> Drei von fünf ist doch nicht schlecht :)

Zwei von fünf, denn "bit" ist nur hier so definiert, während das
(daher Peters Bezugnahme) woanders tatsächlich einen eigenständigen
Typ darstellt.  Da hilft dir <stdint.h> gar nicht (und existiert
möglicherweise auf dem alten 8051-Compiler nicht einmal).

Da sich die Frage jedoch auf die leeren #defines bezog, geht dein
Einwand zu 100 % am Thema vorbei.

[Mark B. (markbrandis)]

Na dann eben zwei von fünf.

Mag auch sein, dass der hier gepostete Code sogar vor C99 entstanden 
ist. Code, zumal C-Code, ist verdammt langlebig. Sehe ich auf der Arbeit 
immer wieder. Und nicht immer zu meiner Freude ;-)

[Bernd N (Gast)]

Google mal nach Speichermodel 8051.

The idata memory type may be used to declare variables only. You may not 
declare idata functions. This memory is indirectly accessed using 8-bit 
addresses and is the on-chip RAM of the 8051. The amount of idata is 
limited in size (to 256 bytes or less). The lower addresses of idata 
overlap the corresponding addresses of data memory.

Variables declared idata are located in the IDATA memory class.

[Peter D. (peda)]

Mark Brandis schrieb:
> Mag auch sein, dass der hier gepostete Code sogar vor C99 entstanden
> ist.

Meine Keil C51 Version ist von 1995.
Läuft nur noch in der DOSBox.