From 36fcc1b98def3fb6e20cf5e877ffc3c1592d0140 Mon Sep 17 00:00:00 2001 From: Robert Moore Date: Fri, 6 Nov 2015 07:25:00 -0800 Subject: Debugger: Remove some unecessary NULL checks local strupr function already checks for NULL pointers. Original linux patch submitted by: Markus Elfring --- source/components/debugger/dbcmds.c | 11 ++--------- source/components/debugger/dbinput.c | 5 +---- 2 files changed, 3 insertions(+), 13 deletions(-) diff --git a/source/components/debugger/dbcmds.c b/source/components/debugger/dbcmds.c index 805dc7577..280e135ce 100644 --- a/source/components/debugger/dbcmds.c +++ b/source/components/debugger/dbcmds.c @@ -1355,15 +1355,8 @@ AcpiDbTrace ( UINT32 Flags = 0; - if (EnableArg) - { - AcpiUtStrupr (EnableArg); - } - - if (OnceArg) - { - AcpiUtStrupr (OnceArg); - } + AcpiUtStrupr (EnableArg); + AcpiUtStrupr (OnceArg); if (MethodArg) { diff --git a/source/components/debugger/dbinput.c b/source/components/debugger/dbinput.c index e1bfa4865..a051e78a3 100644 --- a/source/components/debugger/dbinput.c +++ b/source/components/debugger/dbinput.c @@ -754,10 +754,7 @@ AcpiDbGetLine ( /* Uppercase the actual command */ - if (AcpiGbl_DbArgs[0]) - { - AcpiUtStrupr (AcpiGbl_DbArgs[0]); - } + AcpiUtStrupr (AcpiGbl_DbArgs[0]); Count = i; if (Count) -- cgit v1.2.1