Fix IWYU in fxcrt headers.
authorTom Sepez <tsepez@chromium.org>
Mon, 6 Apr 2015 19:28:23 +0000 (12:28 -0700)
committerTom Sepez <tsepez@chromium.org>
Mon, 6 Apr 2015 19:28:23 +0000 (12:28 -0700)
Confirmed manually that:
  g++ --std=c++0x -o /dev/null -I. ... -c fx_xxxxx.h
is now error-free for the files in this directory. Also kill
some ifndef's around inclusion since we know this doesn't
provide benefit with modern compilers.

R=thestig@chromium.org

Review URL: https://codereview.chromium.org/1064433005

core/include/fxcrt/fx_arb.h
core/include/fxcrt/fx_basic.h
core/include/fxcrt/fx_coordinates.h
core/include/fxcrt/fx_ext.h
core/include/fxcrt/fx_memory.h
core/include/fxcrt/fx_stream.h
core/include/fxcrt/fx_string.h
core/include/fxcrt/fx_ucd.h
core/include/fxcrt/fx_xml.h

index a7b25c1..f173ee7 100644 (file)
@@ -6,6 +6,9 @@
 
 #ifndef _FX_ARABIC_
 #define _FX_ARABIC_
+
+#include "fx_system.h"
+
 class IFX_BidiChar
 {
 public:
index a3cf585..45a5a6e 100644 (file)
@@ -6,18 +6,11 @@
 
 #ifndef _FX_BASIC_H_
 #define _FX_BASIC_H_
-#ifndef _FX_SYSTEM_H_
-#include "fx_system.h"
-#endif
-#ifndef _FX_MEMORY_H_
+
 #include "fx_memory.h"
-#endif
-#ifndef _FX_STRING_H_
-#include "fx_string.h"
-#endif
-#ifndef _FX_STREAM_H_
 #include "fx_stream.h"
-#endif
+#include "fx_string.h"
+#include "fx_system.h"
 
 // The FX_ArraySize(arr) macro returns the # of elements in an array arr.
 // The expression is a compile-time constant, and therefore can be
index 15d3a64..a9afc57 100644 (file)
@@ -6,6 +6,9 @@
 
 #ifndef _FXCRT_COORDINATES_
 #define _FXCRT_COORDINATES_
+
+#include "fx_basic.h"
+
 template<class baseType> class CFX_PSVTemplate;
 template<class baseType> class CFX_VTemplate;
 template<class baseType> class CFX_PRLTemplate;
index 03ec488..3db772a 100644 (file)
@@ -6,26 +6,17 @@
 
 #ifndef _FXCRT_EXTENSION_
 #define _FXCRT_EXTENSION_
-#ifndef _FX_BASIC_H_
+
+#include "fx_arb.h"
 #include "fx_basic.h"
-#endif
-#ifndef _FXCRT_COORDINATES_
 #include "fx_coordinates.h"
-#endif
-#ifndef _FX_XML_H_
-#include "fx_xml.h"
-#endif
-#ifndef _FX_UNICODE_
 #include "fx_ucd.h"
-#endif
-#ifndef _FX_ARABIC_
-#include "fx_arb.h"
-#endif
+#include "fx_xml.h"
+
 #ifdef __cplusplus
 extern "C" {
 #endif
 
-
 FX_FLOAT               FXSYS_tan(FX_FLOAT a);
 FX_FLOAT               FXSYS_logb(FX_FLOAT b, FX_FLOAT x);
 FX_FLOAT               FXSYS_strtof(FX_LPCSTR pcsStr, FX_INT32 iLength = -1, FX_INT32 *pUsedLen = NULL);
@@ -33,6 +24,7 @@ FX_FLOAT              FXSYS_wcstof(FX_LPCWSTR pwsStr, FX_INT32 iLength = -1, FX_INT32 *pUsed
 FX_LPWSTR              FXSYS_wcsncpy(FX_LPWSTR dstStr, FX_LPCWSTR srcStr, size_t count);
 FX_INT32               FXSYS_wcsnicmp(FX_LPCWSTR s1, FX_LPCWSTR s2, size_t count);
 FX_INT32               FXSYS_strnicmp(FX_LPCSTR s1, FX_LPCSTR s2, size_t count);
+
 inline FX_BOOL FXSYS_islower(FX_INT32 ch)
 {
     return ch >= 'a' && ch <= 'z';
@@ -50,8 +42,6 @@ inline FX_INT32 FXSYS_toupper(FX_INT32 ch)
     return ch < 'a' || ch > 'z' ? ch : (ch - 0x20);
 }
 
-
-
 FX_DWORD       FX_HashCode_String_GetA(FX_LPCSTR pStr, FX_INT32 iLength, FX_BOOL bIgnoreCase = FALSE);
 FX_DWORD       FX_HashCode_String_GetW(FX_LPCWSTR pStr, FX_INT32 iLength, FX_BOOL bIgnoreCase = FALSE);
 
index cb71c04..15dc7da 100644 (file)
@@ -6,9 +6,9 @@
 
 #ifndef _FX_MEMORY_H_
 #define _FX_MEMORY_H_
-#ifndef _FX_SYSTEM_H_
+
 #include "fx_system.h"
-#endif
+
 #ifdef __cplusplus
 extern "C" {
 #endif
index b378450..6732789 100644 (file)
@@ -6,9 +6,10 @@
 
 #ifndef _FX_STREAM_H_
 #define _FX_STREAM_H_
-#ifndef _FX_MEMORY_H_
+
 #include "fx_memory.h"
-#endif
+#include "fx_string.h"
+
 void* FX_OpenFolder(FX_LPCSTR path);
 void* FX_OpenFolder(FX_LPCWSTR path);
 FX_BOOL FX_GetNextFile(void* handle, CFX_ByteString& filename, FX_BOOL& bFolder);
index e4546a0..ad1d083 100644 (file)
@@ -6,6 +6,9 @@
 
 #ifndef _FX_STRING_H_
 #define _FX_STRING_H_
+
+#include "fx_memory.h"
+
 class CFX_ByteStringC;
 class CFX_ByteString;
 class CFX_WideStringC;
index 9e1862a..abf953d 100644 (file)
@@ -6,6 +6,9 @@
 
 #ifndef _FX_UNICODE_
 #define _FX_UNICODE_
+
+#include "fx_system.h"
+
 enum FX_CHARBREAKPROP {
     FX_CBP_OP = 0,
     FX_CBP_CL = 1,
index fb46a93..8fb0015 100644 (file)
@@ -6,9 +6,9 @@
 
 #ifndef _FX_XML_H_
 #define _FX_XML_H_
-#ifndef _FX_BASIC_H_
+
 #include "fx_basic.h"
-#endif
+
 class CXML_AttrItem : public CFX_Object
 {
 public: