1884009 - tools/profiler/public/GeckoTraceEvent.h:835:13: error: unused variable 'num_args' [-Werror,-Wunused-variable]

Status: RESOLVED FIXED

(Core :: Gecko Profiler, defect)

Description

[Mike Hommey [:glandium]]

When building with --enable-project=mobile/ios --enable-warnings-as-errors:

 0:02.37 /tmp/gecko-ios/tools/profiler/public/GeckoTraceEvent.h:835:13: error: unused variable 'num_args' [-Werror,-Wunused-variable]
 0:02.37   835 |   const int num_args = 1;
 0:02.38       |             ^~~~~~~~
 0:02.38 /tmp/gecko-ios/tools/profiler/public/GeckoTraceEvent.h:851:13: error: unused variable 'num_args' [-Werror,-Wunused-variable]
 0:02.38   851 |   const int num_args = 2;
 0:02.38       |             ^~~~~~~~
 0:02.38 /tmp/gecko-ios/tools/profiler/public/GeckoTraceEvent.h:852:15: error: unused variable 'arg_names' [-Werror,-Wunused-variable]
 0:02.38   852 |   const char* arg_names[2] = {arg1_name, arg2_name};
 0:02.38       |               ^~~~~~~~~

Comment 1

[Mike Hommey [:glandium]]

--disable-gecko-profiler --enable-warnings-as-errors would yield the same error.

Comment 2

[Mike Hommey [:glandium]]

Attachment: Bug 1884009 - Avoid warnings about unused arguments when building with profiler disabled.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/3a7538c9b373
Avoid warnings about unused arguments when building with profiler disabled. r=padenot

Comment 4

[Serban Stanca [:SerbanS]]

https://hg.mozilla.org/mozilla-central/rev/3a7538c9b373