Changeset 67350 in vbox for trunk/src/VBox/Devices/EFI/Firmware/VBoxPkg/Library/VBoxDebugLib/VBoxPrintHexDump.c

Timestamp:

Jun 12, 2017 5:42:14 PM (8 years ago)

Author:

vboxsync

Message:

EFI/Firmware: fix compiler confusion caused by using ASMIntDisableFlags which resulted in truncated EFI debug output, plus some minor cleanup

File:

• trunk/src/VBox/Devices/EFI/Firmware/VBoxPkg/Library/VBoxDebugLib/VBoxPrintHexDump.c (modified) (2 diffs)

trunk/src/VBox/Devices/EFI/Firmware/VBoxPkg/Library/VBoxDebugLib/VBoxPrintHexDump.c

@@ -30 +30 @@
 *   Header Files                                                               *
 *******************************************************************************/
+#include <ctype.h>
 #include "VBoxDebugLib.h"
 #include "DevEFI.h"
-#include <iprt/ctype.h>
+#include "iprt/asm.h"
 
 
@@ -80 +81 @@
         cchPrinted += VBoxPrintString("  ");
         for (i = 0; i < 16 && i < cb; i++)
-            cchPrinted += vboxPrintHexDumpChar(pb[i] == ' '
-                                               ? ' '
-                                               : RT_C_IS_GRAPH(pb[i])
+            cchPrinted += vboxPrintHexDumpChar(isprint(pb[i])
                                                ? pb[i]
                                                : '.');