Last Comment Bug 650162 - "arm.cpp:165: warning: 'bool<unnamed>::check_edsp()' defined but not used" -- same for "check_armv6()" & "check_neon()"
: "arm.cpp:165: warning: 'bool<unnamed>::check_edsp()' defined but not used" --...
Product: Core
Classification: Components
Component: XPCOM (show other bugs)
: Trunk
: x86_64 Linux
-- normal (vote)
: mozilla6
Assigned To: Timothy B. Terriberry (:derf)
: Nathan Froyd [:froydnj]
Depends on: 767034
Blocks: 583958
  Show dependency treegraph
Reported: 2011-04-14 16:57 PDT by Daniel Holbert [:dholbert]
Modified: 2012-06-21 10:17 PDT (History)
2 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

#ifdef out unused functions (4.04 KB, patch)
2011-04-14 19:02 PDT, Timothy B. Terriberry (:derf)
dholbert: review+
Details | Diff | Splinter Review

Description User image Daniel Holbert [:dholbert] 2011-04-14 16:57:28 PDT
xpcom/glue/arm.cpp:165: warning: 'bool<unnamed>::check_edsp()' defined but not used
xpcom/glue/arm.cpp:171: warning: 'bool<unnamed>::check_armv6()' defined but not used
xpcom/glue/arm.cpp:177: warning: 'bool<unnamed>::check_neon()' defined but not used

Looks like these functions are defined always (or at least "on linux x86_64 debug builds"), but their callers are #ifdef'd like e.g.

> 192 #if !defined(MOZILLA_PRESUME_ARMV6)
> 193     bool armv6_enabled = check_armv6();
> 194 #endif

Could we add the same #if conditions around these function definitions, to avoid building them when we don't need them, and to avoid spamming compile warnings?
Comment 1 User image Daniel Holbert [:dholbert] 2011-04-14 16:58:46 PDT
actually, the more important ifdef around these functions' usages is probably:
Comment 2 User image Timothy B. Terriberry (:derf) 2011-04-14 19:02:53 PDT
Created attachment 526173 [details] [diff] [review]
#ifdef out unused functions

I refactored the #ifdef's to be consistent about when things are defined vs. called, and also updated the indentation (which was inconsistent) and added comments for the closing #endifs which were far from their corresponding #ifs.
Comment 3 User image :Ms2ger (⌚ UTC+1/+2) 2011-04-16 05:46:57 PDT

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