Как делают я лучше всего заставляю предупреждение замолчать о неиспользуемых переменных?

Я видел это вместо (void)param2 способ заставить предупреждение замолчать:

void foo(int param1, int param2)
{
    std::ignore = param2;
    bar(param1);
}

Похож, это было добавлено в C++ 11

Использование UNREFERENCED_PARAMETER (p) может работать. Я знаю, что он определен в WinNT.h для систем Windows и может быть легко определен и для gcc (если он еще не установлен).

НЕДОПУСТИМЫЙ ПАРАМЕТР (p) определяется как

#define UNREFERENCED_PARAMETER(P)          (P)

в WinNT.h.

Не безопасно ли всегда закомментировать имена параметров? Если это не так, вы можете сделать что-нибудь вроде

#ifdef _MSC_VER
# define P_(n) n
#else
# define P_(n)
#endif

void ProcessOps::sendToExternalApp(
    QString sAppName, QString sImagePath,
    qreal P_(qrLeft), qreal P_(qrTop), qreal P_(qrWidth), qreal P_(qrHeight))

Это немного менее уродливо.

Я не вижу вашей проблемы с предупреждением. Укажите в заголовке метода / функции, что компилятор xy выдаст здесь (правильное) предупреждение, но эти переменные необходимы для платформы z.

Предупреждение правильное, отключать его не нужно. Это не делает программу недействительной - но это должно быть документально подтверждено, что на это есть причина.

Во-первых, предупреждение генерируется определением переменной в исходном файле, а не в файле заголовка. Заголовок может оставаться нетронутым и должен, поскольку вы можете использовать что-то вроде doxygen для создания документации API.

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

Пример:

func(int a, int b)
{
    b;
    foo(a);
}

Это может показаться загадочным, поэтому определен макрос вроде UNUSED. MFC сделал это следующим образом:

#ifdef _DEBUG
#define UNUSED(x)
#else
#define UNUSED(x) x
#endif

Вы видите, что предупреждение все еще находится в отладочных сборках, может быть полезно.

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

Пример:

func(int a, int b)
{
    b;
    foo(a);
}

Это может показаться загадочным, поэтому определен макрос вроде UNUSED. MFC сделал это следующим образом:

#ifdef _DEBUG
#define UNUSED(x)
#else
#define UNUSED(x) x
#endif

Как это вы видите, предупреждение все еще находится в отладочных сборках, может быть полезно.

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

Пример:

func(int a, int b)
{
    b;
    foo(a);
}

Это может показаться загадочным, поэтому определен макрос вроде UNUSED. MFC сделал это следующим образом:

#ifdef _DEBUG
#define UNUSED(x)
#else
#define UNUSED(x) x
#endif

Как это вы видите, предупреждение все еще находится в отладочных сборках, может быть полезно.

Ваше текущее решение является лучшим - закомментируйте имя параметра, если вы его не используете. Это применимо ко всем компиляторам, поэтому вам не нужно использовать препроцессор, чтобы сделать это специально для GCC.

В GCC и Clang вы можете использовать директиву препроцессора __ __ ((unused)) для достижения ваша цель.
Например:

int foo (__attribute__((unused)) int bar) {
   return 0;
}

В большинстве случаев использование директив препроцессора считается злом. В идеале вы хотите избегать их, как вредителя. Помните, что заставить компилятор понять ваш код легко, а позволить другим программистам понять ваш код намного сложнее. Несколько десятков подобных случаев тут и там делают очень трудным чтение для себя позже или для других прямо сейчас.

Одним из способов может быть объединение ваших параметров в какой-то класс аргументов. Затем вы можете использовать только подмножество переменных (что эквивалентно действительному присвоению 0) или иметь разные специализации этого класса аргументов для каждой платформы. Однако это может не стоить того, вам нужно проанализировать, подойдет ли он.

Если вы умеете читать невозможные шаблоны, вы можете найти дополнительные советы в книге «Исключительный C ++». Если бы люди, которые читали ваш код, могли бы получить свой набор навыков, позволяющий охватить сумасшедшие вещи, изложенные в этой книге, у вас был бы красивый код, который также можно было бы легко прочитать. Компилятор также будет хорошо осведомлен о том, что вы делаете (вместо того, чтобы скрывать все с помощью предварительной обработки)

Сотрудник только что указал мне на этот красивый маленький макрос здесь

Для простоты я включу макрос ниже.

#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

void dcc_mon_siginfo_handler(int UNUSED(whatsig))