There is nothing wrong with either practice. The GP is just stating their preferred style.

In fact it is good practice to use struct wrappers for void* pointers to get type safety. On the other hand, a typedef is just for programmer convenience and the compiler doesn't care.