Changeset 42688

Timestamp:

04/20/09 15:32:24 (11 months ago)

Author:

beidson@apple.com

Message:

2009-04-20 Brady Eidson < beidson@apple.com>

Rubberstamped by Mark Rowe

Clean builds are broken on case-sensitive filesystems. Let's fix, shall we?

• platform/graphics/SimpleFontData.cpp: Remove "String.h" include. Ojan probably intended to include "PlatformString.h" which was unnecessary because it is pulled in through other headers. This wasn't a build failure on case-insensitive file systems because those systems would find the system <string.h>, averting the warning.

Location:

trunk/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/graphics/SimpleFontData.cpp (modified) (2 diffs)

trunk/WebCore/ChangeLog

@@ +1 @@
+2009-04-20  Brady Eidson  <beidson@apple.com>
+
+        Rubberstamped by Mark Rowe
+
+        Clean builds are broken on case-sensitive filesystems.  Let's fix, shall we?
+
+        * platform/graphics/SimpleFontData.cpp: Remove "String.h" include.  Ojan probably intended to include
+          "PlatformString.h" which was unnecessary because it is pulled in through other headers.  This wasn't a 
+          build failure on case-insensitive file systems because those systems would find the system <string.h>, averting
+          the warning.
+
 2009-04-20  Geoffrey Garen  <ggaren@apple.com>
 

trunk/WebCore/platform/graphics/SimpleFontData.cpp

@@ -33 +33 @@
 #include "Font.h"
 #include "FontCache.h"
+
 #if ENABLE(SVG_FONTS)
 #include "SVGFontData.h"
@@ -38 +39 @@
 #include "SVGFontFaceElement.h"
 #include "SVGGlyphElement.h"
-#include "String.h"
-
 #endif