Question

hello everyone I have this snippet of the code:

void writer(void* param){
    if(NULL == param){
        return;
    }
    param = (param_t*)param;
...
}

is it safe code, or not, param is from type param_t*, but I’m sick doing every time casting when I use it in function, do Somebody have another elegant solution? thanks in advance

Answer 1

That is a strange no-op.

When you define the function you say param is of type void*.
Then, with the cast, you explicitly convert it to param_t*
And the compiler implicitly converts that param_t* to void* with the assignment.

You need another variable

void writer(void *param) {
    param_t *internal_param;

    if (NULL == param) return;
    internal_param = param;
    /* ... */
}