Re: [PATCH v5] Add systemd socket launching support

[Matthew Leach]

Hi Paul,

Matthew Leach <address@hidden> writes:
> Paul Eggert <address@hidden> writes:

[...]

>> @@ -7775,7 +7772,7 @@ init_process_emacs (void)
>>    FD_ZERO (&non_keyboard_wait_mask);
>>    FD_ZERO (&non_process_wait_mask);
>>    FD_ZERO (&write_mask);
>> -  max_process_desc = max_input_desc = -1;
>> +  max_process_desc = max_input_desc = external_sock_fd = -1;
>
> Setting `external_sock_fd' here doesn't work as 'init_process_emacs' is
> called later in 'main' than 'set_external_socket_descriptor' is; any
> descriptor that is passed through from systemd would be overwritten with
> -1.

Should I send in a patch to fix this or are you OK doing it?

Thanks,
-- 
Matt