Re: BUG #16059: Tab-completion of filenames in COPY commands removes required quotes

On 2020-01-14 02:38, Tom Lane wrote:
> I wrote:
>> Peter Eisentraut <peter.eisentraut@2ndquadrant.com> writes:
>>> I have found a weird behavior with identifier quoting, which is not the
>>> subject of this patch, but it appears to be affected by it.
> 
>> I'll think about how to improve this.  Given that we're dequoting
>> filenames explicitly anyway, maybe we don't need to tell readline that
>> double-quote is a quoting character.  Another idea is that maybe
>> we ought to refactor things so that identifier dequoting and requoting
>> is handled explicitly, similarly to the way filenames work now.
>> That would make the patch a lot bigger though.
> 
> On reflection, it seems like the best bet for the moment is to
> remove double-quote from the rl_completer_quote_characters string,
> which should restore all behavior around double-quoted strings to
> what it was before.  (We have to keep single-quote in that string,
> though, or quoted file names fail entirely.)

This patch (version 6) looks good to me.

-- 
Peter Eisentraut              http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services