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
Add extension namespace macro #6676
Add extension namespace macro #6676
Conversation
I didn't replace all namespace-qualified strings using the macro, but happy to do that if you think that this is a useful change. |
d632837
to
c8cb4d0
Compare
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #6676 +/- ##
==========================================
+ Coverage 80.06% 81.50% +1.43%
==========================================
Files 190 191 +1
Lines 37181 36417 -764
Branches 9450 9464 +14
==========================================
- Hits 29770 29682 -88
+ Misses 2997 2971 -26
+ Partials 4414 3764 -650 ☔ View full report in Codecov by Sentry. |
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.
LGTM
c8cb4d0
to
51e7bb7
Compare
Replaced remaining cases (I think I got all of them). |
c5cb295
to
58cc048
Compare
There is a mix of macros for extension namespace and hard-coded extension name in the code. This commit adds an `EXTENSION_NAMESPACE` macro to `extension_constants.h` and uses that instead of locally defined macros and hard-coded strings. It also adds a macro `EXTENSION_OPTION` which can be used to create a full name for an extension option using the namespace macro.
58cc048
to
de84a9e
Compare
There is a mix of macros for extension namespace and hard-coded extension name in the code. This commit adds an
EXTENSION_NAMESPACE
macro toextension_constants.h
and uses that instead of locally defined macros and hard-coded strings. It also adds a macroEXTENSION_OPTION
which can be used to create a full name for an extension option using the namespace macro.Disable-check: force-changelog-file, loader-change