Skip to content

Commit 51dbc87

Browse files
committed
Add C comment about why older compilers complain about basebackup.c's
longjump.
1 parent 895ad83 commit 51dbc87

File tree

1 file changed

+6
-0
lines changed

1 file changed

+6
-0
lines changed

src/backend/replication/basebackup.c

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -217,6 +217,12 @@ perform_base_backup(basebackup_options *opt, DIR *tblspcdir)
217217
ptr.xlogid = logid;
218218
ptr.xrecoff = logseg * XLogSegSize + TAR_SEND_SIZE * i;
219219

220+
/*
221+
* Some old compilers, e.g. 2.95.3/x86, think that passing
222+
* a struct in the same function as a longjump might clobber
223+
* a variable. bjm 2011-02-04
224+
* http://lists.apple.com/archives/xcode-users/2003/Dec//msg00051.html
225+
*/
220226
XLogRead(buf, ptr, TAR_SEND_SIZE);
221227
if (pq_putmessage('d', buf, TAR_SEND_SIZE))
222228
ereport(ERROR,

0 commit comments

Comments
 (0)