On 2025-04-07 Mo 8:25 AM, Mahendra Singh Thalor wrote:
Hi,
In commit 643a1a61985bef2590496, we did some cleanup and we replaced if-else with switch case. Basically, we made a function to open a directory in pg_dumpall. In pg_backup_directory.c file also, we are opening a directory with if-else.
Here, I am attaching a patch which makes both the files similar.

We can move this similar function into one common file also but as of now, I made a static function same as pg_dumpall.c.


Yeah, let's put it in a common file. There's no point in duplicating it.


cheers


andrew



--
Andrew Dunstan
EDB: https://www.enterprisedb.com



Reply via email to