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

[Kevin Rodgers]

Kai Großjohann wrote:

> 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.)

Objection!  (I've always wanted to say that :-)

On the basis of unnecessary cons'ing.

--
<a href="mailto:&lt;kevin.rodgers&#64;ihs.com&gt;";>Kevin Rodgers</a>