Bug 30473

Summary: Check COMPILER(MSVC) instead of PLATFORM(WIN_OS) to avoid the use of vasprintf.
Product: WebKit Reporter: Kwang Yul Seo <skyul>
Component: DOMAssignee: Kwang Yul Seo <skyul>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, aroben, cdumez, eric, mrowe
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Windows XP   
Attachments:
Description Flags
Use COMPILER(MSVC) guards to avoid vasprintf eric: review+, eric: commit-queue-

Description Kwang Yul Seo 2009-10-16 20:28:39 PDT
vasprintf is missing in MSVC. Use COMPILER(MSVC) guards instead of PLALTFORM(WIN_OS) guards.
Comment 1 Kwang Yul Seo 2009-10-16 20:30:16 PDT
Created attachment 41349 [details]
Use COMPILER(MSVC) guards to avoid vasprintf
Comment 2 Eric Seidel (no email) 2009-10-16 23:37:32 PDT
So you're saying that vasprintf works fine in gcc (or any other compiler) on WIN_OS?
Comment 3 Kwang Yul Seo 2009-10-17 00:36:26 PDT
I use MSVC to build WebKit for BREW simulator and I undefined WTF_PLATFORM WIN_OS in Platform.h in favor of WTF_PLATFORM_BREW. vasprintf is not related to WIN_OS but to MSVC.

vasprintf works fine in cygwin or mingw gcc anyway.
Comment 4 Eric Seidel (no email) 2009-10-19 13:46:04 PDT
I could have sworn bdash commented on this bug already, but I don't see it.
Comment 5 Eric Seidel (no email) 2009-10-19 13:47:59 PDT
Comment on attachment 41349 [details]
Use COMPILER(MSVC) guards to avoid vasprintf

+        No new tests. (OOPS!)

the OOPS will cause the commit to fail.  The one in "reviewed by" is OK, because the commit-queue will substitute in the right thing.  BUt the no new tests line should be replaced by something explaingin why this does not need testing or why testing is impossible.

Something simple like:

Build fix only, no tests possible.

would be fine.

cq- since the commit-queue can't handle this as-is.  If you post a new patch  w/o the second OOPS! that can be r+/cq+'d though.
Comment 6 Adam Barth 2009-10-19 21:39:36 PDT
Committed r49838: <http://trac.webkit.org/changeset/49838>
Comment 7 Lucas Forschler 2019-02-06 09:04:19 PST
Mass moving XML DOM bugs to the "DOM" Component.