On Fri, Jan 18, 2019 at 3:13 PM Andres Freund <and...@anarazel.de> wrote:

> TestForOldSnapshot()

> to me it seems wrong from a layering POV to have this in
> bufmgr.[ch]. Due to the inline functions this makes bufmgr.h have a
> dependency on snapmgr.h and tqual.h, which to me seems wrong from a
> layer POV.  Why isn't this declared in snapmgr.h or a new header file?

I will take a look.

-- 
Kevin Grittner
VMware vCenter Server
https://www.vmware.com/

Reply via email to