On Tue, Oct 9, 2012 at 4:34 AM, Daniel Shahaf <d...@daniel.shahaf.name> wrote:
> Hyrum K Wright wrote on Mon, Oct 08, 2012 at 21:51:02 -0400:
>> Poking around subversion/libsvn_subr/gpg_agent.c I found this snippet
>> (at around line 325):
>>
>> [[[
>> /* Send DISPLAY to the gpg-agent daemon. */
>> display = getenv("DISPLAY");
>> if (display != NULL)
>> {
>> request = apr_psprintf(pool, "OPTION display=%s\n", display);
>> if (!send_option(sd, buffer, BUFFER_SIZE, "display", display, pool))
> ...
>>
>> I don't know enough about what's going on here. Is the first time the
>> variable is set supposed to be combined with the second, or is it just
>> a superfluous assignment which we can safely remove?
>>
>
> Remove it. The definition of send_option() recomputes REQUEST and sends
> it over the socket --- the copy here is not needed.
r1396534
-Hyrum
