NotificationsYou must be signed in to change notification settings
Fork28
Star151

Commit28ae524

committed

Quieten warnings about unused variables

These variables are only ever written to in assertion-enabled builds,and the latest Microsoft compilers complain about such variables innon-assertion-enabled builds.Apparently they don't worry so much about variables that are written tobut not read from, so most of our PG_USED_FOR_ASSERTS_ONLY variablesdon't cause the problem.Discussion:https://postgr.es/m/7800.1505950322@sss.pgh.pa.us

1 parent9140cf8 commit28ae524Copy full SHA for 28ae524

File tree

1 file changed

-11

lines changed

src/backend/optimizer/path
- costsize.c

1 file changed

-11

lines changed

`‎src/backend/optimizer/path/costsize.c`

Lines changed: 2 additions & 11 deletions

Original file line number	Diff line number	Diff line change
`@@ -4550,15 +4550,11 @@ set_subquery_size_estimates(PlannerInfo root, RelOptInfo rel)`
`4550`	`4550`	`{`
`4551`	`4551`	`PlannerInfo*subroot=rel->subroot;`
`4552`	`4552`	`RelOptInfo*sub_final_rel;`
`4553`		`-RangeTblEntry*rtePG_USED_FOR_ASSERTS_ONLY;`
`4554`	`4553`	`ListCell*lc;`
`4555`	`4554`
`4556`	`4555`	`/* Should only be applied to base relations that are subqueries */`
`4557`	`4556`	`Assert(rel->relid>0);`
`4558`		`-#ifdefUSE_ASSERT_CHECKING`
`4559`		`-rte=planner_rt_fetch(rel->relid,root);`
`4560`		`-Assert(rte->rtekind==RTE_SUBQUERY);`
`4561`		`-#endif`
	`4557`	`+Assert(planner_rt_fetch(rel->relid,root)->rtekind==RTE_SUBQUERY);`
`4562`	`4558`
`4563`	`4559`	`/*`
`4564`	`4560`	`* Copy raw number of output rows from subquery. All of its paths should`
`@@ -4670,14 +4666,9 @@ set_function_size_estimates(PlannerInfo root, RelOptInfo rel)`
`4670`	`4666`	`void`
`4671`	`4667`	`set_tablefunc_size_estimates(PlannerInforoot,RelOptInforel)`
`4672`	`4668`	`{`
`4673`		`-RangeTblEntry*rtePG_USED_FOR_ASSERTS_ONLY;`
`4674`		`-`
`4675`	`4669`	`/* Should only be applied to base relations that are functions */`
`4676`	`4670`	`Assert(rel->relid>0);`
`4677`		`-#ifdefUSE_ASSERT_CHECKING`
`4678`		`-rte=planner_rt_fetch(rel->relid,root);`
`4679`		`-Assert(rte->rtekind==RTE_TABLEFUNC);`
`4680`		`-#endif`
	`4671`	`+Assert(planner_rt_fetch(rel->relid,root)->rtekind==RTE_TABLEFUNC);`
`4681`	`4672`
`4682`	`4673`	`rel->tuples=100;`
`4683`	`4674`

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!

Commit28ae524

File tree

1 file changed

1 file changed

`‎src/backend/optimizer/path/costsize.c`

0 commit comments