[WINCE] Don't define WTF_CPU_MIDDLE_ENDIAN=1

[Kwang Yul Seo]

Windows CE supports little-endian format only, so don't define WTF_CPU_MIDDLE_ENDIAN=1. Otherwise, it causes a crash in dtoa.

[Kwang Yul Seo]

Created attachment 53145 [details]
Patch

MSDN mentions that Windows CE supports little-endian format only.

http://msdn.microsoft.com/en-us/library/ms905093.aspx

[Laszlo Gombos]

Should we define __ARM_EABI__ for WinCE instead ?

[Kwang Yul Seo]

(In reply to comment #2)
> Should we define __ARM_EABI__ for WinCE instead ?

We can add __ARM_EABI__ to the preprocessor definition in MSVC project or vsprops. However, as we don't have a build system for WinCE, I think it is better to keep the guard here. It seems Android does the same thing.

Who defines __ARM_EABI__ in other platforms?

[Patrick R. Gansterer]

I did the same patch in my working copy too, but didn't find the MSDN entry about the little-endian format ;-).
I think this patch is ok since ANDROID does the same "hack" and we don't have a WinCE buildsystem in the moment.

[Laszlo Gombos]

Comment on attachment 53145 [details]
Patch

r=me.

[WebKit Commit Bot]

Comment on attachment 53145 [details]
Patch

Clearing flags on attachment: 53145

Committed r57804: <http://trac.webkit.org/changeset/57804>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.