rdelaage
/
postgresql
mirror of https://git.postgresql.org/git/postgresql.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Use int instead of size_t in procarray.c. 

All size_t variables declared in procarray.c are actually int ones.
Let's use int instead of size_t for those variables. Which would
reduce Wsign-compare compiler warnings.

Back-patch to v14 where commit 941697c3c1 added size_t variables
in procarray.c, to make future back-patching easy though
this patch is classified as refactoring only.

Reported-by: Ranier Vilela
Author: Ranier Vilela, Aleksander Alekseev
https://postgr.es/m/CAEudQAqyoTZC670xWi6w-Oe2_Bk1bfu2JzXz6xRfiOUzm7xbyQ@mail.gmail.com
This commit is contained in:
Fujii Masao 2021-09-16 12:52:30 +09:00
parent 0c39c29207
commit 64a62ebeeb
1 changed files with 7 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
src/backend/storage/ipc/procarray.c
View File

@ -714,7 +714,7 @@ ProcArrayEndTransaction(PGPROC *proc, TransactionId latestXid)
static inline void
ProcArrayEndTransactionInternal(PGPROC *proc, TransactionId latestXid)
{
size_t pgxactoff = proc->pgxactoff;
int pgxactoff = proc->pgxactoff;
/*
* Note: we need exclusive lock here because we're going to change other
@ -886,7 +886,7 @@ ProcArrayGroupClearXid(PGPROC *proc, TransactionId latestXid)
void
ProcArrayClearTransaction(PGPROC *proc)
{
size_t pgxactoff;
int pgxactoff;
/*
* Currently we need to lock ProcArrayLock exclusively here, as we
@ -1366,7 +1366,7 @@ TransactionIdIsInProgress(TransactionId xid)
TransactionId topxid;
TransactionId latestCompletedXid;
int mypgxactoff;
size_t numProcs;
int numProcs;
int j;
/*
@ -1443,7 +1443,7 @@ TransactionIdIsInProgress(TransactionId xid)
/* No shortcuts, gotta grovel through the array */
mypgxactoff = MyProc->pgxactoff;
numProcs = arrayP->numProcs;
for (size_t pgxactoff = 0; pgxactoff < numProcs; pgxactoff++)
for (int pgxactoff = 0; pgxactoff < numProcs; pgxactoff++)
{
int pgprocno;
PGPROC *proc;
@ -2209,7 +2209,7 @@ GetSnapshotData(Snapshot snapshot)
TransactionId *other_xids = ProcGlobal->xids;
TransactionId xmin;
TransactionId xmax;
size_t count = 0;
int count = 0;
int subcount = 0;
bool suboverflowed = false;
FullTransactionId latest_completed;
@ -2291,7 +2291,7 @@ GetSnapshotData(Snapshot snapshot)
if (!snapshot->takenDuringRecovery)
{
size_t numProcs = arrayP->numProcs;
int numProcs = arrayP->numProcs;
TransactionId *xip = snapshot->xip;
int *pgprocnos = arrayP->pgprocnos;
XidCacheStatus *subxidStates = ProcGlobal->subxidStates;
@ -2301,7 +2301,7 @@ GetSnapshotData(Snapshot snapshot)
* First collect set of pgxactoff/xids that need to be included in the
* snapshot.
*/
for (size_t pgxactoff = 0; pgxactoff < numProcs; pgxactoff++)
for (int pgxactoff = 0; pgxactoff < numProcs; pgxactoff++)
{
/* Fetch xid just once - see GetNewTransactionId */
TransactionId xid = UINT32_ACCESS_ONCE(other_xids[pgxactoff]);