Sorry, I missed this one last night. On Sat, Apr 05, 2025 at 02:10:10AM -0400, Tom Lane wrote: > =?utf-8?Q?=C3=81lvaro?= Herrera <alvhe...@alvh.no-ip.org> writes: >> ... AFAICS we could move both DefnDumperPtr and >> DataDumperPtr typedefs to pg_backup_archiver.h, together with struct >> _tocEntry (the only place where they are used), and we'd have less of a >> mess here. Then we don't need struct TocEntry in pg_backup.h anymore. > > +1 for this simplification.
This is a better fix. Thanks for taking care of it. -- nathan