Subject: Re: Feature improvement of tab completion for DEALLOCATE
To: btnakamichin <btnakamichin@oss.nttdata.com>, pgsql-hackers@postgresql.org
References: <ec1a45b06edfce13706f2c765778d8c2@oss.nttdata.com>
From: Fujii Masao <masao.fujii@oss.nttdata.com>
Message-ID: <997a84af-c2e2-abe4-aca2-abdfb7383346@oss.nttdata.com>
Date: Fri, 25 Sep 2020 14:30:37 +0900
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.14; rv:68.0)
 Gecko/20100101 Thunderbird/68.12.0
MIME-Version: 1.0
In-Reply-To: <ec1a45b06edfce13706f2c765778d8c2@oss.nttdata.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 8bit
Precedence: bulk


On 2020/09/25 13:45, btnakamichin wrote:
> Hello!
> 
> I’d like to improve the deallocate tab completion.
> 
> The deallocate tab completion can’t complement “ALL”. Therefore, this patch fixes the problem.

Thanks for the patch! It looks basically good, but I think it's better
to add newline just before UNION to avoid long line, as follows.

-               COMPLETE_WITH_QUERY(Query_for_list_of_prepared_statements);
+               COMPLETE_WITH_QUERY(Query_for_list_of_prepared_statements
+                                                       " UNION SELECT 'ALL'");

Regards,

-- 
Fujii Masao
Advanced Computing Technology Center
Research and Development Headquarters
NTT DATA CORPORATION