Closed Bug 758140 Opened 13 years ago Closed 13 years ago

Build break for debug build

Categories

(Core :: Widget, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

()

RESOLVED DUPLICATE of bug 758093

People

(Reporter: cyu, Assigned: cyu)

Details

Attachments

(1 file)

Add ac_add_options --enable-debug to gonk-misc/default-gecko-config and build fails in OrientationSensor.cpp: ../../dist/include/nsTSubstring.h:52: note: the mangling of 'va_list' has changed in GCC 4.4 /home/cervantes/hg/mozilla-central/widget/gonk/OrientationObserver.cpp: In function 'nsresult<unnamed>::ConvertToScreenRotation(mozilla::dom::ScreenOrientation, PRUint32*)': /home/cervantes/hg/mozilla-central/widget/gonk/OrientationObserver.cpp:98: warning: comparison between signed and unsigned integer expressions /home/cervantes/hg/mozilla-central/widget/gonk/OrientationObserver.cpp: In function 'nsresult<unnamed>::ConvertToDomOrientation(PRUint32, mozilla::dom::ScreenOrientation*)': /home/cervantes/hg/mozilla-central/widget/gonk/OrientationObserver.cpp:124: warning: comparison between signed and unsigned integer expressions /home/cervantes/hg/mozilla-central/widget/gonk/OrientationObserver.cpp: In member function 'virtual void OrientationObserver::Notify(const mozilla::hal::SensorData&)': /home/cervantes/hg/mozilla-central/widget/gonk/OrientationObserver.cpp:178: error: 'SensorType' has not been declared /home/cervantes/hg/mozilla-central/widget/gonk/OrientationObserver.cpp: In member function 'void OrientationObserver::EnableAutoOrientation()': /home/cervantes/hg/mozilla-central/widget/gonk/OrientationObserver.cpp:240: error: 'AutoOrientationEnabled' was not declared in this scope /home/cervantes/hg/mozilla-central/widget/gonk/OrientationObserver.cpp: In member function 'bool OrientationObserver::LockScreenOrientation(mozilla::dom::ScreenOrientation)': /home/cervantes/hg/mozilla-central/widget/gonk/OrientationObserver.cpp:261: error: 'aOrientations' was not declared in this scope
Attachment #626723 - Flags: review?(mwu)
Attachment #626723 - Flags: review?(mwu) → review+
Keywords: checkin-needed
Patch doesn't apply, because it looks like bug 758093 fixed this same thing. Duping.
Status: NEW → RESOLVED
Closed: 13 years ago
Keywords: checkin-needed
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: