Closed Bug 863804 Opened 7 years ago Closed 7 years ago

Fix some noisy warnings in Android build

Categories

(Core :: JavaScript Engine, defect, P4)

ARM
Android
defect

Tracking

()

RESOLVED FIXED
mozilla23
Tracking Status
firefox22 --- wontfix
firefox23 --- fixed

People

(Reporter: cpeterson, Assigned: cpeterson)

Details

Attachments

(2 files)

Fix -Wsign-compare warning in MacroAssemblerARM::transferMultipleByRunsImpl(). This warning is in a header file so it is very noisy.

js/src/ion/arm/MacroAssembler-arm.h:387:14 [-Wsign-compare] comparison between signed and unsigned integer expressions
Attachment #739695 - Flags: review?(dvander)
Fix -Wunused-but-set-variable warnings in NunboxAssembler.h. These warnings are in a header file, so they are very noisy.

js/src/methodjit/NunboxAssembler.h:192:21 [-Wunused-but-set-variable] variable 'start' set but not used
js/src/methodjit/NunboxAssembler.h:218:21 [-Wunused-but-set-variable] variable 'start' set but not used
js/src/methodjit/NunboxAssembler.h:243:21 [-Wunused-but-set-variable] variable 'start' set but not used
Assignee: general → cpeterson
Status: NEW → ASSIGNED
Attachment #739696 - Flags: review?(dvander)
Attachment #739695 - Flags: review?(dvander) → review+
Attachment #739696 - Flags: review?(dvander) → review+
https://hg.mozilla.org/mozilla-central/rev/d715a456d2cc
https://hg.mozilla.org/mozilla-central/rev/f7526aecfa5b
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla23
You need to log in before you can comment on or make changes to this bug.