New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Honor usage of OidIsValid() macro #4908
Honor usage of OidIsValid() macro #4908
Conversation
5f65b16
to
77986f2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have added a minor comment, the rest looks good to me.
06c651f
to
baec18c
Compare
Codecov Report
@@ Coverage Diff @@
## main #4908 +/- ##
===========================================
- Coverage 89.50% 63.04% -26.47%
===========================================
Files 225 226 +1
Lines 50238 44682 -5556
===========================================
- Hits 44968 28168 -16800
- Misses 5270 16514 +11244
Continue to review full report at Codecov.
|
f822815
to
c24619e
Compare
c24619e
to
81fc8ad
Compare
Postgres source code define the macro `OidIsValid()` to check if the Oid is valid or not (comparing against the `InvalidOid` type). See `src/include/c.h` in Postgres source three. Changed all direct comparisons against `InvalidOid` for the `OidIsValid` call and add a coccinelle check to make sure the future changes will use it correctly.
81fc8ad
to
2ef09cb
Compare
Postgres source code define the macro
OidIsValid()
to check if the Oid is valid or not (comparing against theInvalidOid
type). Seesrc/include/c.h
in Postgres source three.Changed all direct comparisons against
InvalidOid
for theOidIsValid
call and add a coccinelle check to make sure the future changes will use it correctly.