Closed Bug 501864 Opened 10 years ago Closed 10 years ago

Symbian emulator should defines as i386, not ARM.

Categories

(NSPR :: NSPR, defect, P1)

x86
Symbian
defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: m_kato, Assigned: m_kato)

Details

Attachments

(1 file)

Attached patch patch v1Splinter Review
Symbian emulator (WINS / WINSCW) is i386 architecture, not ARM.  But current NSPR defines as ARM.  It should not define as it.
Attachment #386437 - Flags: review?(wtc)
Comment on attachment 386437 [details] [diff] [review]
patch v1

After reviewing the patch, I see that the issue here is *not* with any symbol
that is defined (definition is created) by NSPR, but rather the issue is 
that NSPR only tests for the presence of __WINS__ when __arm__ is also defined, 
rather than treating those symbols as mutually exclusive.

The patch changes the behavior so that effectively __WINS__ is an alternative 
to, and mutually exclusive with, __arm__.  Since this change only affects 
files that that are only built in symbian builds, I have no objection to 
this change.  r=nelson
Attachment #386437 - Flags: review?(wtc) → review+
pr/include/md/_symbian.cfg; new revision: 1.4; previous revision: 1.3
pr/include/md/_symbian.h;   new revision: 1.2; previous revision: 1.1
Assignee: wtc → m_kato
Status: NEW → RESOLVED
Closed: 10 years ago
Priority: -- → P1
Resolution: --- → FIXED
Target Milestone: --- → 4.8.1
You need to log in before you can comment on or make changes to this bug.