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

[Kai Großjohann]

Kevin Rodgers <address@hidden> writes:

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

It seems you do not object to the unification, just to my choice of
(a)?

It seems that setenv does quite a bit of work in addition to consing
stuff onto process-environment.  I'm not sure it is a good idea to
skip using setenv.  Wasn't there a discussion about having setenv do
something additional to frobbing process-environment?  It was some
moons ago.  I'm not talking about the multibyte issue that was
recently mentioned.  Hmmm...
-- 
A preposition is not a good thing to end a sentence with.