Commit6499008

committed

pg_dump: Add const decorations

Add const decorations to the *info arguments of the dump* functions,to clarify that they don't modify that argument. Many other nearbyfunctions modify their arguments, so this can help clarify thesedifferent APIs a bit.Discussion:https://www.postgresql.org/message-id/flat/012d3030-9a2c-99a1-ed2d-988978b5632f%40enterprisedb.com

1 parent4ad5611 commit6499008Copy full SHA for 6499008

File tree

+176

-176

lines changed

+176

-176

lines changed

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -248,7 +248,7 @@ typedef int DumpId;`
`248`	`248`	`* Function pointer prototypes for assorted callback methods.`
`249`	`249`	`*/`
`250`	`250`
`251`		`-typedefint (DataDumperPtr) (ArchiveAH,void*userArg);`
	`251`	`+typedefint (DataDumperPtr) (ArchiveAH,constvoid*userArg);`
`252`	`252`
`253`	`253`	`typedefvoid (SetupWorkerPtrType) (ArchiveAH);`
`254`	`254`

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -382,7 +382,7 @@ struct _tocEntry`
`382`	`382`	`intnDeps;/* number of dependencies */`
`383`	`383`
`384`	`384`	`DataDumperPtrdataDumper;/* Routine to dump data for object */`
`385`		`-voiddataDumperArg;/ Arg for above routine */`
	`385`	`+constvoiddataDumperArg;/ Arg for above routine */`
`386`	`386`	`voidformatData;/ TOC Entry data specific to file format */`
`387`	`387`
`388`	`388`	`/* working state while dumping/restoring */`
`@@ -421,7 +421,7 @@ typedef struct _archiveOpts`
`421`	`421`	`constDumpId*deps;`
`422`	`422`	`intnDeps;`
`423`	`423`	`DataDumperPtrdumpFn;`
`424`		`-void*dumpArg;`
	`424`	`+constvoid*dumpArg;`
`425`	`425`	`}ArchiveOpts;`
`426`	`426`	`#defineARCHIVE_OPTS(...) &(ArchiveOpts){__VA_ARGS__}`
`427`	`427`	`/* Called to add a TOC entry */`

Comments

(0)