From eb060db71b629d72107e67a5057bf70f7d53307d Mon Sep 17 00:00:00 2001 From: Dean Camera Date: Mon, 9 Aug 2010 12:04:52 +0000 Subject: Tighten up the code in the delta movement detection code in the MouseHostWithParser and JoystickHostWithParser demos. Make the ClassDriver JoystickHostWithParser demo use the HID_ALIGN_DATA() macro instead of manual shifting and casting. --- .../JoystickHostWithParser/JoystickHostWithParser.c | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) (limited to 'Demos/Host/ClassDriver/JoystickHostWithParser') diff --git a/Demos/Host/ClassDriver/JoystickHostWithParser/JoystickHostWithParser.c b/Demos/Host/ClassDriver/JoystickHostWithParser/JoystickHostWithParser.c index f1b4cac7b..8420d05ab 100644 --- a/Demos/Host/ClassDriver/JoystickHostWithParser/JoystickHostWithParser.c +++ b/Demos/Host/ClassDriver/JoystickHostWithParser/JoystickHostWithParser.c @@ -148,16 +148,13 @@ int main(void) (ReportItem->Attributes.Usage.Usage == USAGE_Y)) && (ReportItem->ItemType == REPORT_ITEM_TYPE_In)) { - int16_t DeltaMovement = (int16_t)(ReportItem->Value << (16 - ReportItem->Attributes.BitSize)); + int16_t DeltaMovement = HID_ALIGN_DATA(ReportItem, int16_t); - if (ReportItem->Attributes.Usage.Usage == USAGE_X) + if (DeltaMovement) { - if (DeltaMovement) + if (ReportItem->Attributes.Usage.Usage == USAGE_X) LEDMask |= ((DeltaMovement > 0) ? LEDS_LED1 : LEDS_LED2); - } - else - { - if (DeltaMovement) + else LEDMask |= ((DeltaMovement > 0) ? LEDS_LED3 : LEDS_LED4); } } -- cgit v1.2.3