Fix to Tom Briden's recent touch-screen commit; the original commit

[refind] / refind / menu.c

diff --git a/refind/menu.c b/refind/menu.c
index 08abe3e778e54fbbef970db302b784aca67c26ef..fe4cf7db193dd01123cb0d2dab9cbd59d988e40b 100644 (file)
--- a/refind/menu.c
+++ b/refind/menu.c
@@ -394,9 +394,11 @@ static UINTN RunGenericMenu(IN REFIT_MENU_SCREEN *Screen, IN MENU_STYLE_FUNC Sty
     CHAR16 TimeoutMessage[256];
     CHAR16 KeyAsString[2];
     UINTN MenuExit;
     CHAR16 TimeoutMessage[256];
     CHAR16 KeyAsString[2];
     UINTN MenuExit;

-       
-       EFI_STATUS TouchStatus = EFI_NOT_READY;
-       EFI_ABSOLUTE_POINTER_STATE TouchState;
+    EFI_STATUS TouchStatus = EFI_NOT_READY;
+    EFI_ABSOLUTE_POINTER_STATE TouchState;
+    UINT32 TouchScreenPosX;
+    UINT32 TouchScreenPosY;
+    UINTN Item;

 
     if (Screen->TimeoutSeconds > 0) {
         HaveTimeout = TRUE;
 
     if (Screen->TimeoutSeconds > 0) {
         HaveTimeout = TRUE;


@@ -619,11 +621,9 @@ static UINTN RunGenericMenu(IN REFIT_MENU_SCREEN *Screen, IN MENU_STYLE_FUNC Sty
             }
         } else { //react to touch event
             //the TouchProtocol min/max may not match the screen size
             }
         } else { //react to touch event
             //the TouchProtocol min/max may not match the screen size

-            UINT32 TouchScreenPosX = (TouchState.CurrentX * UGAWidth) / TouchProtocol->Mode->AbsoluteMaxX;
-            UINT32 TouchScreenPosY = (TouchState.CurrentY * UGAHeight) / TouchProtocol->Mode->AbsoluteMaxY;
-
-
-            UINTN Item = FindMainMenuItem(Screen, &State, TouchScreenPosX, TouchScreenPosY);
+            TouchScreenPosX = ((UINT32) TouchState.CurrentX * UGAWidth) / (UINT32) TouchProtocol->Mode->AbsoluteMaxX;
+            TouchScreenPosY = ((UINT32) TouchState.CurrentY * UGAHeight) / (UINT32) TouchProtocol->Mode->AbsoluteMaxY;
+            Item = FindMainMenuItem(Screen, &State, TouchScreenPosX, TouchScreenPosY);

             switch (Item) {
                 case TOUCH_NO_ITEM:
                     //do nothing
             switch (Item) {
                 case TOUCH_NO_ITEM:
                     //do nothing