[R] Unnecesary code?

[hunsyntesat at hush.com]

Dear R-ers,

While browsing the R sources, I found the following piece of code 
in src\main\memory.c:

static void reset_pp_stack(void *data)
{
    R_size_t *poldpps = data;
    R_PPStackSize =  *poldpps;
}

To me, it looks like the poldpps pointer is a nuissance; can't you 
just cast the data pointer and derefer it at once? Say,

static void reset_pp_stack(void *data)
{
    R_PPStackSize = * (R_size_t *) data;
}

-- Hun