From: Andres Freund <andres@anarazel.de>
Date: Thu, 30 May 2019 20:44:38 +0000 (-0700)
Subject: Remove unnecessary (and wrong) forward declaration.
X-Git-Tag: REL_12_BETA2~95
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=13002bf0bcdf4900987684ca6ef7665a25aa692b;p=postgresql

Remove unnecessary (and wrong) forward declaration.

Interestingly only C++ compilers have, so far, complained about this
odd forward declaration. This originated when IndexBuildCallback was
defined in another file, but now is completely unnecessary (but was
wrong before too, cpluspluscheck just wouldn't have noticed).

Reported-By: Tom Lane
Discussion: https://postgr.es/m/53941.1559239260@sss.pgh.pa.us
---

diff --git a/src/include/access/tableam.h b/src/include/access/tableam.h
index bf3a472018..af1dcfb8e2 100644
--- a/src/include/access/tableam.h
+++ b/src/include/access/tableam.h
@@ -33,7 +33,6 @@ extern bool synchronize_seqscans;
 
 struct BulkInsertStateData;
 struct IndexInfo;
-struct IndexBuildCallback;
 struct SampleScanState;
 struct TBMIterateResult;
 struct VacuumParams;