diff options
| author | Robert Moore <Robert.Moore@intel.com> | 2015-11-06 07:25:00 -0800 |
|---|---|---|
| committer | Robert Moore <Robert.Moore@intel.com> | 2015-11-06 07:25:00 -0800 |
| commit | 36fcc1b98def3fb6e20cf5e877ffc3c1592d0140 (patch) | |
| tree | 3fcc93e9573b275beb554ddb56b5fd13188b7428 | |
| parent | 5be7dc4d0d69b2953d156f5bc4d3e8a65a390837 (diff) | |
| download | acpica-36fcc1b98def3fb6e20cf5e877ffc3c1592d0140.tar.gz | |
Debugger: Remove some unecessary NULL checks
local strupr function already checks for NULL pointers.
Original linux patch submitted by:
Markus Elfring <elfring@users.sourceforge.net>
| -rw-r--r-- | source/components/debugger/dbcmds.c | 11 | ||||
| -rw-r--r-- | 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) |