Last Comment Bug 772210 - Fix warning: unused variable 'initFlushCount'
: Fix warning: unused variable 'initFlushCount'
Product: Core
Classification: Components
Component: JavaScript Engine (show other bugs)
: Trunk
: ARM Android
-- minor (vote)
: mozilla16
Assigned To: Chris Peterson [:cpeterson]
: Jason Orendorff [:jorendorff]
Depends on:
  Show dependency treegraph
Reported: 2012-07-09 13:50 PDT by Chris Peterson [:cpeterson]
Modified: 2012-07-10 15:49 PDT (History)
2 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

fix-initFlushCount-warning.patch (1.36 KB, patch)
2012-07-09 13:56 PDT, Chris Peterson [:cpeterson]
luke: review+
Details | Diff | Splinter Review

Description User image Chris Peterson [:cpeterson] 2012-07-09 13:50:59 PDT
When building for Android/JS_CPU_ARM, I see many instances of the following compiler warning because BaseAssembler.h is #included in many other files:

mozilla/inbound/js/src/methodjit/BaseAssembler.h: In member function 'JSC::AbstractMacroAssembler<JSC::ARMAssembler>::Call js::mjit::Assembler::callWithABI(void*, bool)':
mozilla/inbound/js/src/methodjit/BaseAssembler.h:563: warning: unused variable 'initFlushCount'
Comment 1 User image Chris Peterson [:cpeterson] 2012-07-09 13:56:29 PDT
Created attachment 640350 [details] [diff] [review]

Fix warning: unused variable 'initFlushCount' by using DebugOnly<int>. This code only affects #ifdef JS_CPU_ARM builds.
Comment 2 User image Chris Peterson [:cpeterson] 2012-07-09 14:16:48 PDT
Comment 3 User image Ryan VanderMeulen [:RyanVM] 2012-07-09 17:30:00 PDT
Don't resolve bugs until they land on m-c.
Comment 4 User image Ryan VanderMeulen [:RyanVM] 2012-07-10 15:49:42 PDT

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