Remove unnecessary (and wrong) forward declaration.

anarazel · anarazel · commit 13002bf0bcdf · 2019-05-30T13:44:38.000-07:00
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
@@ -33,7 +33,6 @@ extern bool synchronize_seqscans;
 
 struct BulkInsertStateData;
 struct IndexInfo;
-struct IndexBuildCallback;
 struct SampleScanState;
 struct TBMIterateResult;
 struct VacuumParams;
