On Wed, Jun 28, 2023 at 03:09:55PM +0800, Richard Guo wrote: > +1. To nitpick, how about we remove the blank line just before removing > the key for top level entry? > > - /* Also remove entries for top level statements */ > + /* Also remove entries if exist for top level statements */ > key.toplevel = true; > - > - /* Remove the key if exists */ > entry = (pgssEntry *) hash_search(pgss_hash, &key, HASH_REMOVE, NULL);
Why not if it improves the overall situation. Could you send a patch with everything you have in mind? -- Michael
signature.asc
Description: PGP signature
