Last Comment Bug 776849 - nsHistory.cpp:242:12: warning: variable ‘rv’ set but not used [-Wunused-but-set-variable]
: nsHistory.cpp:242:12: warning: variable ‘rv’ set but not used [-Wunused-but-s...
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: History: Global (show other bugs)
: Trunk
: All All
: -- trivial (vote)
: mozilla17
Assigned To: :Aryeh Gregor (away until August 15)
:
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-07-24 01:43 PDT by :Aryeh Gregor (away until August 15)
Modified: 2012-08-02 19:10 PDT (History)
1 user (show)
ayg: in‑testsuite-
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
Patch (1.08 KB, patch)
2012-07-24 01:45 PDT, :Aryeh Gregor (away until August 15)
jonas: review+
Details | Diff | Review

Description :Aryeh Gregor (away until August 15) 2012-07-24 01:43:26 PDT
I happen to randomly notice this sometimes while compiling, and I figure I may as well fix it when I have nothing else to do until the full recompile finishes.
Comment 1 :Aryeh Gregor (away until August 15) 2012-07-24 01:45:47 PDT
Created attachment 645226 [details] [diff] [review]
Patch

Picked a reviewer out of my hat.  I'll push it to try along with something else so as not to waste try resources.
Comment 2 :Aryeh Gregor (away until August 15) 2012-07-24 03:30:01 PDT
Pushed to try with another patch: https://tbpl.mozilla.org/?tree=Try&rev=ddeecf8aff59
Comment 3 :Aryeh Gregor (away until August 15) 2012-08-02 02:33:08 PDT
https://hg.mozilla.org/integration/mozilla-inbound/rev/430c7512acee
Comment 4 Ryan VanderMeulen [:RyanVM] 2012-08-02 19:10:02 PDT
https://hg.mozilla.org/mozilla-central/rev/430c7512acee

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