Re: Janitorial issues: Man-getpage-in-background

[Stefan Monnier]

> The function Man-getpage-in-background uses two different ways to set
> environment variables:
> 
> (a) (let ((process-environment (copy-sequence process-environment)))
>       (setenv "FOO" "bar")
>       ...)
> 
> (b) (let ((process-environment (cons "FOO=bar" process-environment)))
>       ...)
> 
> It looks really strange to see them both in the same function...
> 
> Okay to unify?  (I'd choose (a) unless there are objections.)

I'd choose (b) to avoid unnecessary consing (via copying).


        Stefan