dom/base/TimeoutManager.cpp:149:13: error: unused variable 'now' [-Werror,-Wunused-variable]
Categories
(Core :: Performance, defect)
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr60 | --- | unaffected |
firefox65 | --- | unaffected |
firefox66 | --- | fix-optional |
firefox67 | --- | fixed |
People
(Reporter: jbeich, Unassigned)
References
Details
(Keywords: regression)
Attachments
(1 obsolete file)
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•6 years ago
|
||
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•6 years ago
|
||
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
Updated•6 years ago
|
Updated•6 years ago
|
Description
•