[R] Unnecesary code?

[Duncan Murdoch]

hunsyntesat at hush.com wrote:
> 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;
> }
>   
What would you gain by this change?

Duncan Murdoch