## c't-Projekte - Mailinglisten

[Voriger (Datum)] [Nächster (Datum)] [Voriger (Thread)] [Nächster (Thread)]
[Nach Datum][Nach Thread]

### Re: [ct-bot] Patch fuer rc5-codes.h / Conrad FB

Absender: Michail Brzitwa
Datum: Do, 23.03.2006 12:22:30
In-reply-to: <44227526.3050501@xxxxxxxx>
References: <441BDB32.2060307@xxxxxx> <200603231050.42022.michail@xxxxxxxxxx> <44227526.3050501@xxxxxxxx>

    Am Donnerstag, 23. März 2006 11:15 schrieb Benjamin Benz:
    > Wir erfassen in Fällen wie ihrem die Taste mit ihrem Label. Dann haben
    > kann sie jeder eindeutig identifizieren. Des weiteren fügen wir in der
    > Definition der FB ein Mapping für die jeweilige Taste ein.
    >
    > Ein Beispiel:
    >
    > Eine FB hat keine PWR-taste aber eine Taste auf der "TV" steht, die sich
    > für die PWR-Funktion eignen würde. Dann steht in der Definition:
    >
    > #define RC5_CODE_TV 	(0x2017 & RC5_MASK)
    > #define RC5_CODE_PWR	RC5_CODE_TV

    Das fände ich ok, obwohl nicht unterschieden wird zwischen Namen,
    die im Code referenziert werden (PWR), und denen, die nur abgebildet
    werden (TV).

    Als wirklich nur minimale Verbesserung würde ich vorschlagen, in rc5.c
    und Konsorten Funktionscodes zu verwenden, und in rc-codes.h sowohl
    Tastencodes als auch Funktionscodes zu definieren. Zum Beispiel:

    #define RC5_CODE_0	(0x3000 & RC5_MASK)
    ...
    #define RC5_CODE_TV	(0x2017 & RC5_MASK)
    ...
    #define RC_FKEY_0	RC5_CODE_0
    #define RC_FKEY_PWR	RC5_CODE_TV

    und in rc5.c/gRemCtrlAction statt RC5_CODE_PWR dann RC_FKEY_PWR zu
    verwenden. Die RC5_CODE_XXXX würden dann nur RC5-Konstanten darstellen,
    und die RC_FKEY_YYYY nur auf RC5_CODE_XXXX verweisen dürfen.

    Das wäre zwar kein echtes Mapping (es kann immer noch gedoppelte Codes
    geben bzw. muß per Kompiliertest herausgefunden werden, ob denn alles
    definiert wurde), aber immerhin etwas.

    Wenn sowas erwünscht ist, kann ich das als Patch testweise umsetzen,
    ist ja nur etwas Schreibarbeit. In rc5-codes.h müsste die Anleitung
    für neue FBs ebenso angepasst werden. Gruß,
    --
    Michail Brzitwa           <michail@xxxxxxxxxx>            +49-511-343215

- Antworten:
  - RE: [ct-bot] Patch fuer rc5-codes.h / Conrad FB
    - From: Timo Sandmann
  - Re: [ct-bot] Patch fuer rc5-codes.h / Conrad FB
    - From: Benjamin Benz

- Referenzen:
  - Re: [ct-bot] Patch fuer rc5-codes.h / Conrad FB
    - From: Michail Brzitwa
  - Re: [ct-bot] Patch fuer rc5-codes.h / Conrad FB
    - From: Benjamin Benz

- Voriger (Datum): Re: [ct-bot] Patch fuer rc5-codes.h / Conrad FB, Benjamin Benz
- Nächster (Datum): RE: [ct-bot] Patch fuer rc5-codes.h / Conrad FB, Timo Sandmann
- Voriger (Thread): Re: [ct-bot] Patch fuer rc5-codes.h / Conrad FB, Benjamin Benz
- Nächster (Thread): RE: [ct-bot] Patch fuer rc5-codes.h / Conrad FB, Timo Sandmann
- Indizes:
  - Datum
  - Thread