Bug 34190 – [BREWMP] Remove COMPILE_ASSERT conflict with the underlying PLATFORM

Bug 34190 - [BREWMP] Remove COMPILE_ASSERT conflict with the underlying PLATFORM


Summary:	[BREWMP] Remove COMPILE_ASSERT conflict with the underlying PLATFORM

Status:	RESOLVED FIXED

Product:	WebKit
Component:	JavaScriptCore
Version:	528+ (Nightly build)
Platform:	Other Other

Importance:	P2 Normal
Assigned To:	Nobody

URL:
Keywords:

Depends on:
Blocks:	33564
	Show dependency tree / graph

Reported:	2010-01-26 16:48 PST by Kwang Yul Seo
Modified:	2010-02-03 18:59 PST (History)

Attachments
Remove COMPILE_ASSERT conflict (1.42 KB, patch) 2010-01-26 16:52 PST, Kwang Yul Seo	no flags	Review Patch \| Details \| Formatted Diff \| Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note

You need to log in before you can comment on or make changes to this bug.

Description From Kwang Yul Seo 2010-01-26 16:48:43 PST

COMPILE_ASSERT conflicts with the underlying PLATFORM because it is defined both in WTF's Assertions.h and BREWMP's AEEClassIDs.h. Include "AEEClassIDs.h" in Assertions.h and undef COMPILE_ASSERT to avoid redefining COMPILE_ASSERT.

------- Comment #1 From Kwang Yul Seo 2010-01-26 16:52:32 PST -------

Created an attachment (id=47462) [details]
Remove COMPILE_ASSERT conflict

------- Comment #2 From Eric Seidel 2010-02-03 13:06:11 PST -------

(From update of attachment 47462 [details])
It seems strange to me to include AEEClassIDs.h

I assume that's where COMPILE_ASSERT is defined?

------- Comment #3 From WebKit Commit Bot 2010-02-03 13:43:43 PST -------

(From update of attachment 47462 [details])
Clearing flags on attachment: 47462

Committed r54296: <http://trac.webkit.org/changeset/54296>

------- Comment #4 From WebKit Commit Bot 2010-02-03 13:43:51 PST -------

All reviewed patches have been landed.  Closing bug.

------- Comment #5 From Kwang Yul Seo 2010-02-03 18:59:00 PST -------

(In reply to comment #2)
> (From update of attachment 47462 [details] [details])
> It seems strange to me to include AEEClassIDs.h
> 
> I assume that's where COMPILE_ASSERT is defined?

Yes. COMPILE_ASSERT is defined in AEEClassIDs.h.