Bug 23598 - wml/WMLTableElement.cpp has non-ASCII characters in comment and can't be compiled on CJK Windows
Summary: wml/WMLTableElement.cpp has non-ASCII characters in comment and can't be comp...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC Windows XP
: P2 Trivial
Assignee: Jungshik Shin
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-01-28 12:58 PST by Jungshik Shin
Modified: 2009-01-29 16:45 PST (History)
2 users (show)

See Also:


Attachments
patch (1.43 KB, patch)
2009-01-28 13:14 PST, Jungshik Shin
ap: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jungshik Shin 2009-01-28 12:58:03 PST
WebCore/wml/WMLTableElement.cpp has non-ASCII characters (0 enclosed by smart-quotes) in UTF-8 and leads to a compilation error (warning treated as an error) on CJK Windows (where 3-byte sequences for smart quotes cannot be interpreted as valid in the default system codepage). Either we have to replace them with ASCII double-quotes or add 'BOM' character represented in UTF-8 at the beginning of a file. I propose the former is better because on other platforms/compiler/build tools, 'BOM' in UTF-8 might be problematic.
Comment 1 Jungshik Shin 2009-01-28 13:14:18 PST
Created attachment 27119 [details]
patch

Applying this patch is a step forward, but there are other header files (in Quicktime SDK and in WebKitLibraries/win/include) with the same issue leading to a compilation error on CJK Windows. For Quicktime SDK, I filed a bug report at bugreport.apple.com a while ago.
Comment 2 Jungshik Shin 2009-01-28 13:17:28 PST
Other files in WebKitLibraries/win/include with the issue are CMTypes.h and CMApplication.h

Perhaps, we have to add a note to the Windows build page about the issue (do not set the Windows system code page to one of CJK codepages to build webkit on Windows) until all of them are cleaned up. 
Comment 3 Alexey Proskuryakov 2009-01-28 13:38:14 PST
Comment on attachment 27119 [details]
patch

r=me
Comment 4 Darin Fisher (:fishd, Google) 2009-01-29 16:45:40 PST
http://trac.webkit.org/changeset/40387