NotificationsYou must be signed in to change notification settings
Fork28
Star151

Commitab87b8f

committed

Mark variable used only in assertion with PG_USED_FOR_ASSERTS_ONLY

Perpendicular lines always intersect, so the line_interpt_line() returnvalue in line_closept_point() was used only in an assertion, triggeringcompiler warnings in non-assert builds.

1 parent74294c7 commitab87b8fCopy full SHA for ab87b8f

File tree

1 file changed

-2

lines changed

src/backend/utils/adt
- geo_ops.c

1 file changed

-2

lines changed

`‎src/backend/utils/adt/geo_ops.c`

Lines changed: 3 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -2528,14 +2528,15 @@ lseg_interpt_line(Point result, LSEG lseg, LINE *line)`
`2528`	`2528`	`staticfloat8`
`2529`	`2529`	`line_closept_point(Pointresult,LINEline,Point*point)`
`2530`	`2530`	`{`
`2531`		`-boolretval;`
	`2531`	`+boolretvalPG_USED_FOR_ASSERTS_ONLY;`
`2532`	`2532`	`Pointclosept;`
`2533`	`2533`	`LINEtmp;`
`2534`	`2534`
`2535`	`2535`	`/* We drop a perpendicular to find the intersection point. */`
`2536`	`2536`	`line_construct(&tmp,point,line_invsl(line));`
`2537`	`2537`	`retval=line_interpt_line(&closept,line,&tmp);`
`2538`		`-Assert(retval);/* XXX: We need something better. */`
	`2538`	`+`
	`2539`	`+Assert(retval);/* perpendicular lines always intersect */`
`2539`	`2540`
`2540`	`2541`	`if (result!=NULL)`
`2541`	`2542`	`*result=closept;`

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!

Commitab87b8f

File tree

1 file changed

1 file changed

`‎src/backend/utils/adt/geo_ops.c`

0 commit comments