On Mon, Mar 14, 2022 at 7:51 AM Dilip Kumar <dilipbal...@gmail.com> wrote: > Other than that, I have fixed some mistakes in comments and supported > tab completion for the new options.
I was looking at 0001 and 0002 again and realized that I swapped the names load_relmap_file() and read_relmap_file() from what I should have done. Here's a revised version. With this, read_relmap_file() and write_relmap_file() become functions that just read and write the file without touching any global variables, and load_relmap_file() is the function that reads data from the file and puts it into a global variable, which seems more sensible than the way I had it before. -- Robert Haas EDB: http://www.enterprisedb.com
relmap-rmh-refactor-v2.patch
Description: Binary data