Подавить предупреждение о неиспользуемой переменной в C++ => ошибка компилятора или ошибка кода?

В настоящее время я использую следующий шаблон функции для подавления неиспользуемых предупреждений переменных:

template<typename T>
void
unused(T const &) {
  /* Do nothing. */
}

однако, при переносе на cygwin из Linux, я теперь получаю ошибки компилятора на g++ 3.4.4 (на linux я 3.4.6, так что, может быть, это исправление ошибки?):

Write.cpp: In member function `void* Write::initReadWrite()':
Write.cpp:516: error: invalid initialization of reference of type 'const volatile bool&' from expression of type 'volatile bool'
../../src/common/Assert.h:27: error: in passing argument 1 of `void unused(const T&) [with T = volatile bool]'
make[1]: *** [ARCH.cygwin/release/Write.o] Error 1

аргумент unused является переменной-членом, объявленной как:

  volatile bool readWriteActivated;

это ошибка компилятора или ошибка в моем коде?

вот минимальный тестовый случай:

template<typename T>
void unused(T const &) { }

int main() {
  volatile bool x = false;
  unused(!x); // type of "!x" is bool
}
11
c++ language-lawyer member-variables templates

5 ответов

фактический способ указать, что вы фактически не используете параметр, не дает ему имени:

int f(int a, float) {
     return a*2;
}

будет компилироваться везде со всеми включенными предупреждениями, без предупреждения о неиспользуемом float. Даже если аргумент имеет имя в прототипе (например,int f(int a, float f);), он все равно не жалуются.

28

Я не на 100% уверен, что это портативный, но это идиома, которую я обычно использовал для подавления предупреждений о неиспользуемых переменных. Контекст здесь-обработчик сигнала, который используется только для catch SIGINT и SIGTERM, поэтому, если функция когда-либо вызывается, я знаю, что пришло время для выхода программы.

volatile bool app_killed = false;
int signal_handler(int signum)
{
    (void)signum; // this suppresses the warnings
    app_killed = true;
}

мне не нравится загромождать список параметров с помощью __attribute__((unused)), так как трюк cast-to-void работает без использования макросов для Visual C++.

9

Это ошибка компилятора, и нет известных обходных путей:

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42655

он фиксируется в v4.4.

4

в GCC можно определить макрос следующим образом:

#ifdef UNUSED
#elif defined(__GNUC__)
# define UNUSED(x) UNUSED_ ## x __attribute__((unused))
#elif defined(__LCLINT__)
# define UNUSED(x) /*@unused@*/ x
#else
# define UNUSED(x) x
#endif

любые параметры, отмеченные этим макросом, будут подавлять неиспользуемые предупреждения GCC (и переименовывают параметр с префиксом UNUSED_). Для Visual Studio можно подавлять предупреждения с помощью

2

ответ, предложенный haavee (с поправками ur), является тем, который я обычно использую:

int f(int a, float /*epsilon*/) {
     return a*2;
}

реальная проблема возникает, когда аргумент иногда но не всегда используется в методе, например:

int f(int a, float epsilon) {
#ifdef LOGGING_ENABLED
     LOG("f: a = %d, epsilon = %f\n", a, epsilon);
#endif
     return a*2;
}

Теперь я не могу прокомментировать имя параметра epsilon, потому что это нарушит мою сборку журнала (я не хочу вставлять другой #ifdef в список аргументов, потому что это делает код намного сложнее читать).

поэтому я думаю, что лучший решением было бы использовать предложение Тома:

int f(int a, float epsilon) {
(void) epsilon;    // suppress compiler warning for possibly unused arg
#ifdef LOGGING_ENABLED
     LOG("f: a = %d, epsilon = %f\n", a, epsilon);
#endif
     return a*2;
}

меня беспокоит только то, что некоторые компиляторы могут предупредить о предупреждении "(void) epsilon;", например, предупреждение "оператор не имеет эффекта" или что - то в этом роде-думаю, мне просто придется протестировать все компиляторы, которые я, вероятно, буду использовать...

1