bug#23063: 24.5; Multi-line sql statements are not saved correctly

[Stefan Kangas]

Stefan Kangas <stefan@marxist.se> writes:

> Hi Michael,
>
> Nikolay Kudryavtsev <nikolay.kudryavtsev@gmail.com> writes:
>
>> Currently comint.el does not respect local values of 
>> comint-input-ring-separator. Sql mode on the other hand sets it locally. 
>> This prevents multi-line sql history from working.
>>
>> Sql history is stored using comint-write-input-ring. When it saves the 
>> history file, it uses with-current-buffer, thus ignoring previous local 
>> value. When read comint-read-input-ring uses with-temp-buffer with similar 
>> results.
>>
>> This seems to have been caused by the fix for this bug.
>
> This bug was tagged as "unreproducible moreinfo" in a message to
> control@debbugs called "Close bugs" sent on 21 Apr 2019, but the bug
> itself was never closed.
>
> Should the bug be closed as well?

No reply here within 14 weeks.  I'm going to assume the intention here
was to close this bug, so I'm doing that now.

If this conclusion is incorrect, please reply to this email (use "Reply
to all" in your email client) and we can reopen the bug report.

Best regards,
Stefan Kangas