Bug 7491

Summary: add new TextEncoding stuff to win32 build
Product: WebKit Reporter: Maciej Stachowiak <mjs>
Component: WebKit Misc.Assignee: Maciej Stachowiak <mjs>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: 420+   
Hardware: PC   
OS: Windows XP   
Attachments:
Description Flags
Add TextEncoding, plus some fixes darin: review+

Maciej Stachowiak
Reported 2006-02-27 03:02:54 PST
add new TextEncoding stuff to win32 build This fixes 7 link errors. There are still two left that seem fairly encoding-related, for converting from utf-16 to a specified encoding. That is because we use CF for encoding.
Attachments
Add TextEncoding, plus some fixes (146.39 KB, patch)
2006-02-27 03:04 PST, Maciej Stachowiak 		darin: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Maciej Stachowiak
Comment 1 2006-02-27 03:04:13 PST
Created attachment 6758 [details] Add TextEncoding, plus some fixes
Darin Adler
Comment 2 2006-02-27 08:19:53 PST
Comment on attachment 6758 [details] Add TextEncoding, plus some fixes std::min is in <algorithm>, so you should include that instead of <utility> in StreamingTextDecoder.cpp. It worries me that the entire WebCore.vcproj is showing up in the diff. Is something configured wrong in Subversion?
Note You need to log in before you can comment on or make changes to this bug.