C secp256k1: What is the purpose of the idiom '(void)data;'?

Question

The following code is contained in the file secp256k1.c:

static void 
default_illegal_callback_fn( 
  const char* str, 
  void* data) 
{
  (void)data;
  fprintf(stderr, "[libsecp256k1] illegal argument: %s\n", str);
  abort();
}

Could someone explain the purpose of the line (void)data;? I am pretty sure there is one, but I can't figure out what it is :(

score 7 · Accepted Answer · answered Jun 03 '17 at 16:09

7

Without (void) data;, gcc will complain about data being an unused variable. This is used throughout the codebase, especially for context objects, to deal with parameters which are required for API/consistency reasons but not actually needed.

answered Jun 03 '17 at 16:09

Andrew Poelstra

1,210
10
8

1

Many compilers also have some variant of __unused(data);, which, if available, is preferred. – abelenky Jun 03 '17 at 18:33

score 3 · Answer 2 · answered Jun 03 '17 at 16:14

3

Notice that without this cast, data would be unused in this function. This is probably here for the purpose of preventing compiler warnings about unused variables, while keeping the argument in the function signature.

answered Jun 03 '17 at 16:14

Jestin

8,812
1
22
32

yes you are right, the cast (void) is necessary to avoid gcc's warning. Thank you ! – Sven Williamson Jun 03 '17 at 16:17

C secp256k1: What is the purpose of the idiom '(void)data;'?

2 Answers2