NotificationsYou must be signed in to change notification settings
Fork27
Star157

Commit5ba8779

Marina Polyakova

committed

Fix compiler warnings due to new checks in PostgreSQL 16

See the commit 0fe954c28584169938e5c0738cfaa9930ce77577 (Add-Wshadow=compatible-local to the standard compilation flags) in PostgreSQL 16.pg_query_state.c: In function ‘pg_query_state’:pg_query_state.c:615:66: warning: declaration of ‘msg’ shadows a previous local[-Wshadow=compatible-local] 615 | shm_mq_msg *msg =(shm_mq_msg *) lfirst(i); | ^~~pg_query_state.c:489:42: note: shadowed declaration is here 489 | shm_mq_msg *msg; | ^~~pg_query_state.c: In function ‘GetRemoteBackendWorkers’:pg_query_state.c:946:25: warning: declaration of ‘proc’ shadows a parameter[-Wshadow=compatible-local] 946 | PGPROC *proc = BackendPidGetProc(pid); | ^~~~pg_query_state.c:913:33: note: shadowed declaration is here 913 | GetRemoteBackendWorkers(PGPROC *proc) | ~~~~~~~~^~~~

1 parent74f6c16 commit5ba8779Copy full SHA for 5ba8779

File tree

1 file changed

-8

lines changed

pg_query_state.c

1 file changed

-8

lines changed

`‎pg_query_state.c‎`

Lines changed: 9 additions & 8 deletions

Original file line number	Diff line number	Diff line change
`@@ -612,17 +612,18 @@ pg_query_state(PG_FUNCTION_ARGS)`
`612`	`612`	`foreach(i,msgs)`
`613`	`613`	`{`
`614`	`614`	`List*qs_stack;`
`615`		`-shm_mq_msgmsg= (shm_mq_msg)lfirst(i);`
	`615`	`+shm_mq_msgcurrent_msg= (shm_mq_msg)lfirst(i);`
`616`	`616`	`proc_statep_state= (proc_state)palloc(sizeof(proc_state));`
`617`	`617`
`618`		`-if (msg->result_code!=QS_RETURNED)`
	`618`	`+if (current_msg->result_code!=QS_RETURNED)`
`619`	`619`	`continue;`
`620`	`620`
`621`		`-AssertState(msg->result_code==QS_RETURNED);`
	`621`	`+AssertState(current_msg->result_code==QS_RETURNED);`
`622`	`622`
`623`		`-qs_stack=deserialize_stack(msg->stack,msg->stack_depth);`
	`623`	`+qs_stack=deserialize_stack(current_msg->stack,`
	`624`	`+current_msg->stack_depth);`
`624`	`625`
`625`		`-p_state->proc=msg->proc;`
	`626`	`+p_state->proc=current_msg->proc;`
`626`	`627`	`p_state->stack=qs_stack;`
`627`	`628`	`p_state->frame_index=0;`
`628`	`629`	`p_state->frame_cursor=list_head(qs_stack);`
`@@ -943,10 +944,10 @@ GetRemoteBackendWorkers(PGPROC *proc)`
`943`	`944`	`for (i=0;i<msg->number;i++)`
`944`	`945`	`{`
`945`	`946`	`pid_tpid=msg->pids[i];`
`946`		`-PGPROC*proc=BackendPidGetProc(pid);`
`947`		`-if (!proc\|\| !proc->pid)`
	`947`	`+PGPROC*current_proc=BackendPidGetProc(pid);`
	`948`	`+if (!current_proc\|\| !current_proc->pid)`
`948`	`949`	`continue;`
`949`		`-result=lcons(proc,result);`
	`950`	`+result=lcons(current_proc,result);`
`950`	`951`	`}`
`951`	`952`
`952`	`953`	`#ifPG_VERSION_NUM<100000`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit5ba8779

File tree

1 file changed

1 file changed

`‎pg_query_state.c‎`

0 commit comments