Re: ldap.el Problem and Solution

[Noah Lavine]

You're right; line 579 makes no sense.

Stefan already installed the fix at line 607 of ldap.el, where it
belongs, so I don't think a patch would be useful here; but in the
future I will try to make sure that I provide a patch against the
trunk.

Thanks,
Noah

On Mon, Oct 29, 2012 at 6:58 AM, Chong Yidong <address@hidden> wrote:
> Noah Lavine <address@hidden> writes:
>
>> The solution is to insert this after line 579 of ldap.el:
>>
>> (if (looking-at "version:")
>>     (forward-line 1))
>>
>> I have tested this, and it seems to work for me.
>
> Please send a patch, preferably against current trunk.  Line 579 of
> ldap.el, on trunk, doesn't seem to be the spot you intended.