Reviewed-by: Jaben Carsey <jaben.car...@intel.com>
> -Original Message-
> From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of
> Ruiyu Ni
> Sent: Tuesday, March 13, 2018 12:41 AM
> To: edk2-devel@lists.01.org
> Cc: Carsey, Jaben <jaben.car...@intel.com>
> Subject: [edk2] [PATCH] ShellPkg/[hex]edit: Fix mouse freeze issue
> Importance: High
>
> In edit or hexedit, the mouse cursor doesn't move when moving
> the mouse.
> The root cause is 5563281fa2b31093a1cbd415553b9264c5136e89
> * ShellPkg/[hex]edit: use SimpleTextInEx to read console
> wrongly uses WaitForEvent() to listen keyboard input.
> It blocks the code execution when there is no keyboard input.
> While the same function also polls the mouse move status,
> the mouse movement cannot be reflected to the screen when
> there is no keyboard input.
>
> The patch fixes the issue by use CheckEvent() instead of
> WaitForEvent().
>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Ruiyu Ni <ruiyu...@intel.com>
> Cc: Jaben Carsey <jaben.car...@intel.com>
> ---
> .../UefiShellDebug1CommandsLib/Edit/MainTextEditor.c | 16 +-
> --
> .../UefiShellDebug1CommandsLib/HexEdit/MainHexEditor.c | 16
> +---
> 2 files changed, 18 insertions(+), 14 deletions(-)
>
> diff --git
> a/ShellPkg/Library/UefiShellDebug1CommandsLib/Edit/MainTextEditor.c
> b/ShellPkg/Library/UefiShellDebug1CommandsLib/Edit/MainTextEditor.c
> index 6832441e81..98e1331ac4 100644
> --- a/ShellPkg/Library/UefiShellDebug1CommandsLib/Edit/MainTextEditor.c
> +++
> b/ShellPkg/Library/UefiShellDebug1CommandsLib/Edit/MainTextEditor.c
> @@ -1840,7 +1840,6 @@ MainEditorKeyInput (
>EFI_KEY_DATA KeyData;
>EFI_STATUSStatus;
>EFI_SIMPLE_POINTER_STATE MouseState;
> - UINTN EventIndex;
>BOOLEAN NoShiftState;
>
>do {
> @@ -1876,8 +1875,11 @@ MainEditorKeyInput (
>}
> }
>
> -Status = gBS->WaitForEvent (1, >WaitForKeyEx,
> );
> -if (!EFI_ERROR (Status) && EventIndex == 0) {
> +//
> +// CheckEvent() returns Success when non-partial key is pressed.
> +//
> +Status = gBS->CheckEvent (MainEditor.TextInputEx->WaitForKeyEx);
> +if (!EFI_ERROR (Status)) {
>Status = MainEditor.TextInputEx->ReadKeyStrokeEx
> (MainEditor.TextInputEx, );
>if (!EFI_ERROR (Status)) {
> //
> @@ -1917,11 +1919,11 @@ MainEditorKeyInput (
> }
>
>}
> - //
> - // after handling, refresh editor
> - //
> - MainEditorRefresh ();
> }
> +//
> +// after handling, refresh editor
> +//
> +MainEditorRefresh ();
>
>} while (Status != EFI_OUT_OF_RESOURCES && !EditorExit);
>
> diff --git
> a/ShellPkg/Library/UefiShellDebug1CommandsLib/HexEdit/MainHexEditor.c
> b/ShellPkg/Library/UefiShellDebug1CommandsLib/HexEdit/MainHexEditor.c
> index a2e52ea39c..1a89d3d72a 100644
> ---
> a/ShellPkg/Library/UefiShellDebug1CommandsLib/HexEdit/MainHexEditor.c
> +++
> b/ShellPkg/Library/UefiShellDebug1CommandsLib/HexEdit/MainHexEditor.c
> @@ -2108,7 +2108,6 @@ HMainEditorKeyInput (
>EFI_KEY_DATA KeyData;
>EFI_STATUSStatus;
>EFI_SIMPLE_POINTER_STATE MouseState;
> - UINTN EventIndex;
>BOOLEAN NoShiftState;
>BOOLEAN LengthChange;
>UINTN Size;
> @@ -2268,8 +2267,11 @@ HMainEditorKeyInput (
>}
> }
>
> -Status = gBS->WaitForEvent (1,
> >WaitForKeyEx, );
> -if (!EFI_ERROR (Status) && EventIndex == 0) {
> +//
> +// CheckEvent() returns Success when non-partial key is pressed.
> +//
> +Status = gBS->CheckEvent (HMainEditor.TextInputEx->WaitForKeyEx);
> +if (!EFI_ERROR (Status)) {
>Status = HMainEditor.TextInputEx->ReadKeyStrokeEx
> (HMainEditor.TextInputEx, );
>if (!EFI_ERROR (Status)) {
> //
> @@ -2351,11 +2353,11 @@ HMainEditorKeyInput (
>}
> }
>}
> - //
> - // after handling, refresh editor
> - //
> - HMainEditorRefresh ();
> }
> +//
> +// after handling, refresh editor
> +//
> +HMainEditorRefresh ();
>
>} while (Status != EFI_OUT_OF_RESOURCES && !HEditorExit);
>
> --
> 2.16.1.windows.1
>
> ___
> edk2-devel mailing list
> edk2-devel@lists.01.org
> https://lists.01.org/mailman/listinfo/edk2-devel
___
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel