On 7 July 2015 at 01:04, Roy Franz <roy.fr...@linaro.org> wrote:
> Treat ASCII 0x7F as backspace, rather than delete, for TTY terminals.  This
> better matches the default Linux terminal settings that are used when 
> connecting
> to a simulated platform using xterm or a similar terminal program.
>
> Signed-off-by: Roy Franz <roy.fr...@linaro.org>
> Contributed-under: TianoCore Contribution Agreement 1.0

Order is wrong ^^^

Other than that
Reviewed-by: Ard Biesheuvel <ard.biesheu...@linaro.org>

> ---
>  MdeModulePkg/Universal/Console/TerminalDxe/TerminalConIn.c | 10 ++++++++--
>  1 file changed, 8 insertions(+), 2 deletions(-)
>
> diff --git a/MdeModulePkg/Universal/Console/TerminalDxe/TerminalConIn.c 
> b/MdeModulePkg/Universal/Console/TerminalDxe/TerminalConIn.c
> index 17a1244..227df85 100644
> --- a/MdeModulePkg/Universal/Console/TerminalDxe/TerminalConIn.c
> +++ b/MdeModulePkg/Universal/Console/TerminalDxe/TerminalConIn.c
> @@ -1561,8 +1561,14 @@ UnicodeToEfiKey (
>      }
>
>      if (UnicodeChar == DEL) {
> -      Key.ScanCode    = SCAN_DELETE;
> -      Key.UnicodeChar = 0;
> +      if (TerminalDevice->TerminalType == TTYTERMTYPE) {
> +        Key.ScanCode    = SCAN_NULL;
> +        Key.UnicodeChar = CHAR_BACKSPACE;
> +      }
> +      else {
> +        Key.ScanCode    = SCAN_DELETE;
> +        Key.UnicodeChar = 0;
> +      }
>      } else {
>        Key.ScanCode    = SCAN_NULL;
>        Key.UnicodeChar = UnicodeChar;
> --
> 2.1.4
>

------------------------------------------------------------------------------
Don't Limit Your Business. Reach for the Cloud.
GigeNET's Cloud Solutions provide you with the tools and support that
you need to offload your IT needs and focus on growing your business.
Configured For All Businesses. Start Your Cloud Today.
https://www.gigenetcloud.com/
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/edk2-devel

Reply via email to