Bug 23598 – wml/WMLTableElement.cpp has non-ASCII characters in comment and can't be compiled on CJK Windows

• Component: WebCore Misc.
• Status: RESOLVED FIXED

[Jungshik Shin]

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.

[Jungshik Shin]

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.

[Jungshik Shin]

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. 

[Alexey Proskuryakov]

Comment on attachment 27119 [details]
patch

r=me

[Darin Fisher (:fishd, Google)]

http://trac.webkit.org/changeset/40387