On 7 July 2015 at 01:04, Roy Franz <[email protected]> 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 <[email protected]>
> Contributed-under: TianoCore Contribution Agreement 1.0

Order is wrong ^^^

Other than that
Reviewed-by: Ard Biesheuvel <[email protected]>

> ---
>  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
[email protected]
https://lists.sourceforge.net/lists/listinfo/edk2-devel

Reply via email to