Jak ukryć ostrzeżenia „nieużywany parametr” w C?

Na przykład:

Bool NullFunc(const struct timespec *when, const char *who)
{
   return TRUE;
}

W C ++ byłem w stanie /*...*/skomentować parametry. Ale oczywiście nie w C, gdzie daje mi to błąd error: parameter name omitted.

Zwykle piszę takie makro:

#define UNUSED(x) (void)(x)

Możesz użyć tego makra do wszystkich nieużywanych parametrów. (Uwaga: działa to na dowolnym kompilatorze).

Na przykład:

void f(int x) {
    UNUSED(x);
    ...
}

W gcc możesz opisać parametr za pomocą unusedatrybutu .

Ten atrybut dołączony do zmiennej oznacza, że ​​zmienna ma być prawdopodobnie nieużywana. GCC nie wyświetli ostrzeżenia dla tej zmiennej.

W praktyce osiąga się to poprzez umieszczenie __attribute__ ((unused))tuż przed parametrem. Na przykład:

void foo(workerid_t workerId) { }

staje się

void foo(__attribute__((unused)) workerid_t workerId) { }

Możesz użyć nieużywanego atrybutu gcc / clang, jednak ja używam tych makr w nagłówku, aby uniknąć posiadania specyficznych atrybutów gcc w całym źródle, również posiadanie __attribute__wszędzie jest trochę gadatliwe / brzydkie.

#ifdef __GNUC__
#  define UNUSED(x) UNUSED_ ## x __attribute__((__unused__))
#else
#  define UNUSED(x) UNUSED_ ## x
#endif

#ifdef __GNUC__
#  define UNUSED_FUNCTION(x) __attribute__((__unused__)) UNUSED_ ## x
#else
#  define UNUSED_FUNCTION(x) UNUSED_ ## x
#endif

Następnie możesz zrobić ...

void foo(int UNUSED(bar)) { ... }

Wolę to, ponieważ pojawia się błąd, jeśli spróbujesz użyć barkodu w dowolnym miejscu, więc nie możesz przez pomyłkę pozostawić atrybutu.

i dla funkcji ...

static void UNUSED_FUNCTION(foo)(int bar) { ... }

Uwaga 1):
O ile mi wiadomo, MSVC nie ma odpowiednika __attribute__((__unused__)).

Uwaga 2): makro nie będzie działać na argumentach, które zawierają nawias, więc jeśli masz argument jak ty nie może zrobić, albo , to jedynym minusem do makro znalazłem do tej pory, w tych przypadkach spadnę z powrotem do
UNUSED
float (*coords)[3]
float UNUSED((*coords)[3])float (*UNUSED(coords))[3]UNUSED(void)coords;

Z gcc z nieużywanym atrybutem:

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

Widząc, że jest to oznaczone jako gcc, możesz użyć przełącznika wiersza poleceń Wno-unused-parameter.

Na przykład:

gcc -Wno-unused-parameter test.c

Oczywiście wpływa to na cały plik (i może projekt, w zależności od ustawienia przełącznika), ale nie musisz zmieniać żadnego kodu.

Specyficznym sposobem gcc / g ++ na wyłączenie ostrzeżenia o nieużywanym parametrze dla bloku kodu źródłowego jest dołączenie go z następującymi instrukcjami pragma:

#pragma GCC diagnostic push
#pragma GCC diagnostic ignored "-Wunused-parameter"
<code with unused parameters here>
#pragma GCC diagnostic pop

Oznaczenie atrybutu jest idealnym sposobem. MAKRO prowadzi do zamieszania. i używając void (x), dodajemy koszty ogólne w przetwarzaniu.

Jeśli nie używasz argumentu wejściowego, użyj

void foo(int __attribute__((unused))key)
{
}

Jeśli nie używasz zmiennej zdefiniowanej w funkcji

void foo(int key)
{
   int hash = 0;
   int bkt __attribute__((unused)) = 0;

   api_call(x, hash, bkt);
}

Teraz później używasz zmiennej hash dla twojej logiki, ale nie potrzebuje bkt. zdefiniuj bkt jako nieużywany, w przeciwnym razie kompilator powie „bkt set bt not used”.

UWAGA: Ma to na celu wyłączenie ostrzeżenia, a nie optymalizację.

Mam ten sam problem. Użyłem biblioteki trzeciej części. Kiedy skompiluję tę bibliotekę, kompilator (gcc / clang) będzie narzekał na nieużywane zmienne.

Lubię to

test.cpp: 29: 11: ostrzeżenie: zmienna „magia” ustawiona, ale nieużywana [-Zużyta-ale-ustawiona-zmienna] krótka magia [] = {

test.cpp: 84: 17: ostrzeżenie: nieużywana zmienna 'before_write' [-Wunused-zmienna] int64_t before_write = Wątek :: currentTimeMillis ();

Więc rozwiązanie jest całkiem jasne. Dodanie -Wno-unusedjako CFCAG gcc / clang spowoduje usunięcie wszystkich „nieużywanych” ostrzeżeń, nawet jeśli są -Wallustawione.

W ten sposób NIE POTRZEBUJESZ zmiany kodu.

W MSVC, aby pominąć określone ostrzeżenie, wystarczy podać jego numer do kompilatora jako / wd #. Mój CMakeLists.txt zawiera taki blok:

If (MSVC)
    Set (CMAKE_EXE_LINKER_FLAGS "$ {CMAKE_EXE_LINKER_FLAGS} / NODEFAULTLIB: LIBCMT")
    Add_definitions (/W4 /wd4512 /wd4702 /wd4100 /wd4510 /wd4355 /wd4127)
    Add_definitions (/D_CRT_SECURE_NO_WARNINGS)
Elseif (CMAKE_COMPILER_IS_GNUCXX OR CMAKE_COMPILER_IS_GNUC)
    Add_definitions (-Wall -W -pedantic)
Else ()
    Message ("Unknown compiler")
Endif ()

Teraz nie mogę powiedzieć, co dokładnie oznaczają / wd4512 / wd4702 / wd4100 / wd4510 / wd4355 / wd4127, ponieważ nie zwracam uwagi na MSVC przez trzy lata, ale tłumią one superantyczne ostrzeżenia, które nie wpływają na wynik.

Widziałem używany ten styl:

if (when || who || format || data || len);

Dla przypomnienia, podoba mi się powyższa odpowiedź Hioba, ale jestem ciekawa rozwiązania, w którym sama nazwa zmiennej byłaby użyta w instrukcji „nic nie robić”:

void foo(int x) {
    x; /* unused */
    ...
}

Jasne, ma to wady; na przykład bez „nieużywanej” nuty wygląda raczej na pomyłkę niż zamierzoną linię kodu.

Korzyścią jest to, że DEFINE nie jest potrzebne i pozbywa się ostrzeżenia.

Czy jest jakaś wydajność, optymalizacja lub inne różnice?