1523506 - dom/base/TimeoutManager.cpp:149:13: error: unused variable 'now' [-Werror,-Wunused-variable]

Status: RESOLVED DUPLICATE of bug 1523333

(Core :: Performance, defect)

Description

[Jan Beich]

GeckoProfiler isn't available on many Tier3 platforms: Linux powerpc/sparc/s390x, BSDs, Solaris.

$ c++ -v
FreeBSD clang version 7.0.1 (tags/RELEASE_701/final 349250) (based on LLVM 7.0.1)
Target: x86_64-unknown-freebsd13.0
Thread model: posix
InstalledDir: /usr/bin

$ echo "ac_add_options --enable-warnings-as-errors # mimic MOZ_AUTOMATION" >>.mozconfig
$ ./mach bootstrap
$ ./mach build
[...]
In file included from objdir/dom/base/Unified_cpp_dom_base5.cpp:110:
dom/base/TimeoutManager.cpp:149:13: error: unused variable 'now' [-Werror,-Wunused-variable]
TimeStamp now;
^
1 error generated.

Comment 1

[(no longer active)]

Attachment: Bug 1523506 - Move the declaration of the now variable to within the MOZ_GECKO_PROFILER block;

Comment 2

[Jan Beich]

Comment on attachment 9039714 [details]
Bug 1523506 - Move the declaration of the now variable to within the MOZ_GECKO_PROFILER block;

I confirm. It unbreaks --enable-warnings-as-errors on FreeBSD.

Comment 3

[Randell Jesup [:jesup] (needinfo me)]

Comment on attachment 9039714 [details]
Bug 1523506 - Move the declaration of the now variable to within the MOZ_GECKO_PROFILER block;

This would break functionality. I'm fixing this as part of bug 1523333