Last Comment Bug 758735 - TestSettingsAPI.cpp:14:20: warning: ‘gFunction’ defined but not used [-Wunused-variable]
: TestSettingsAPI.cpp:14:20: warning: ‘gFunction’ defined but not used [-Wunuse...
Product: Core
Classification: Components
Component: DOM: Device Interfaces (show other bugs)
: Trunk
: x86_64 Linux
: -- normal (vote)
: mozilla15
Assigned To: Daniel Holbert [:dholbert]
: Andrew Overholt [:overholt]
Depends on:
Blocks: buildwarning 743336
  Show dependency treegraph
Reported: 2012-05-25 12:12 PDT by Daniel Holbert [:dholbert]
Modified: 2012-05-26 05:21 PDT (History)
2 users (show)
ryanvm: in‑testsuite-
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

fix (808 bytes, patch)
2012-05-25 12:14 PDT, Daniel Holbert [:dholbert]
anygregor: review+
Details | Diff | Splinter Review

Description Daniel Holbert [:dholbert] 2012-05-25 12:12:21 PDT
xpcom/tests/TestSettingsAPI.cpp:14:20: warning: ‘gFunction’ defined but not used [-Wunused-variable]

This file was added here:
> static const char* gFunction = "main";
as an unused variable.
Comment 1 Daniel Holbert [:dholbert] 2012-05-25 12:14:40 PDT
Created attachment 627316 [details] [diff] [review]
Comment 2 Gregor Wagner [:gwagner] 2012-05-25 12:25:37 PDT
Comment on attachment 627316 [details] [diff] [review]

Oh I thought this is needed for the test-environment. 
Thanks for fixing!
Comment 3 Daniel Holbert [:dholbert] 2012-05-25 12:31:17 PDT
(Nope -- it's used in a different test-file in the same directory just for logging purposes, not for any functional purpose.)

Thanks for the review!
Comment 4 Ryan VanderMeulen [:RyanVM] 2012-05-26 05:21:19 PDT

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