Changeset 209322 in webkit for trunk/Source/JavaScriptCore/disassembler

Timestamp:

Dec 5, 2016, 9:47:34 AM (8 years ago)

Author:

Konstantin Tokarev

Message:

Add STDC_FORMAT_MACROS before inttypes.h is included
​https://bugs.webkit.org/show_bug.cgi?id=165374

We need formatting macros like PRIu64 to be available in all places where
inttypes.h header is used. All these usages get inttypes.h definitions
via wtf/Assertions.h header, except SQLiteFileSystem.cpp where formatting
macros are not used anymore since r185129.

This patch fixes multiple build errors with MinGW and reduces number of
independent STDC_FORMAT_MACROS uses in the code base.

Reviewed by Darin Adler.

Source/JavaScriptCore:

• disassembler/ARM64/A64DOpcode.cpp: Removed STDC_FORMAT_MACROS

because it is obtained via Assertions.h now

• disassembler/ARM64Disassembler.cpp: Ditto.

Source/WebCore:

No new tests needed.

• platform/sql/SQLiteFileSystem.cpp: Removed unused inttypes.h

inclusion.

Source/WTF:

• wtf/Assertions.h: Define STDC_FORMAT_MACROS.
• wtf/StdLibExtras.h: Remove definition of PRId64 for Windows, as we

have STDC_FORMAT_MACROS defined now.

Location:

trunk/Source/JavaScriptCore/disassembler

Files:

• ARM64/A64DOpcode.cpp (modified) (1 diff)
• ARM64Disassembler.cpp (modified) (1 diff)

trunk/Source/JavaScriptCore/disassembler/ARM64/A64DOpcode.cpp

@@ -24 +24 @@
  */
 
-#define __STDC_FORMAT_MACROS
 #include "config.h"
 

trunk/Source/JavaScriptCore/disassembler/ARM64Disassembler.cpp

@@ -24 +24 @@
  */
 
-#define __STDC_FORMAT_MACROS
 #include "config.h"
 #include "Disassembler.h"