NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit65a8a3f

committed

Change various Gin*Is* macros to return 0/1.

Returning the direct result of bit arithmetic, in a macro intended to beused in a boolean manner, can be problematic if the return value isstored in a variable of type 'bool'. If bool is implemented using C99's_Bool, that can lead to comparison failures if the variable is thencompared again with the expression (see ginStepRight() for an examplethat fails), as _Bool forces the result to be 0/1. That happens in someconfigurations of newer MSVC compilers. It's also problematic whenstoring the result of such an expression in a narrower type.Several gin macros have been declared in that style since gin's initialcommit in8a3631f.There's a lot more macros like this, but this is the only one causingregression test failures; and I don't want to commit and backpatch alarger patch with lots of conflicts just before the next set of minorreleases.Discussion: 20150811154237.GD17575@awork2.anarazel.deBackpatch: All supported branches

1 parent8ea71fd commit65a8a3fCopy full SHA for 65a8a3f

File tree

1 file changed

-5

lines changed

src/include/access
- gin_private.h

1 file changed

-5

lines changed

`‎src/include/access/gin_private.h`

Lines changed: 5 additions & 5 deletions

Original file line number	Diff line number	Diff line change
`@@ -106,17 +106,17 @@ typedef struct GinMetaPageData`
`106`	`106`	`*/`
`107`	`107`	`#defineGinPageGetOpaque(page) ( (GinPageOpaque) PageGetSpecialPointer(page) )`
`108`	`108`
`109`		`-#defineGinPageIsLeaf(page) ( GinPageGetOpaque(page)->flags & GIN_LEAF )`
	`109`	`+#defineGinPageIsLeaf(page) ((GinPageGetOpaque(page)->flags & GIN_LEAF) != 0 )`
`110`	`110`	`#defineGinPageSetLeaf(page) ( GinPageGetOpaque(page)->flags \|= GIN_LEAF )`
`111`	`111`	`#defineGinPageSetNonLeaf(page) ( GinPageGetOpaque(page)->flags &= ~GIN_LEAF )`
`112`		`-#defineGinPageIsData(page) ( GinPageGetOpaque(page)->flags & GIN_DATA )`
	`112`	`+#defineGinPageIsData(page) ((GinPageGetOpaque(page)->flags & GIN_DATA) != 0 )`
`113`	`113`	`#defineGinPageSetData(page) ( GinPageGetOpaque(page)->flags \|= GIN_DATA )`
`114`		`-#defineGinPageIsList(page) ( GinPageGetOpaque(page)->flags & GIN_LIST )`
	`114`	`+#defineGinPageIsList(page) ((GinPageGetOpaque(page)->flags & GIN_LIST) != 0 )`
`115`	`115`	`#defineGinPageSetList(page) ( GinPageGetOpaque(page)->flags \|= GIN_LIST )`
`116`		`-#defineGinPageHasFullRow(page) ( GinPageGetOpaque(page)->flags & GIN_LIST_FULLROW )`
	`116`	`+#defineGinPageHasFullRow(page) ((GinPageGetOpaque(page)->flags & GIN_LIST_FULLROW) != 0 )`
`117`	`117`	`#defineGinPageSetFullRow(page) ( GinPageGetOpaque(page)->flags \|= GIN_LIST_FULLROW )`
`118`	`118`
`119`		`-#defineGinPageIsDeleted(page) ( GinPageGetOpaque(page)->flags & GIN_DELETED)`
	`119`	`+#defineGinPageIsDeleted(page) ((GinPageGetOpaque(page)->flags & GIN_DELETED) != 0)`
`120`	`120`	`#defineGinPageSetDeleted(page) ( GinPageGetOpaque(page)->flags \|= GIN_DELETED)`
`121`	`121`	`#defineGinPageSetNonDeleted(page) ( GinPageGetOpaque(page)->flags &= ~GIN_DELETED)`
`122`	`122`

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!

Commit65a8a3f

File tree

1 file changed

1 file changed

`‎src/include/access/gin_private.h`

0 commit comments