-
Notifications
You must be signed in to change notification settings - Fork 500
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
Rename macros in config.h. #257
Conversation
Signed-off-by: Lipeng Zhu <lipeng.zhu@intel.com>
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.
Just a minor comment, Thanks
Signed-off-by: Lipeng Zhu <lipeng.zhu@intel.com>
@hwware @zuiderkwast Part of #25 |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## unstable #257 +/- ##
============================================
- Coverage 68.40% 68.33% -0.08%
============================================
Files 108 108
Lines 61562 61562
============================================
- Hits 42111 42066 -45
- Misses 19451 19496 +45
|
Signed-off-by: Lipeng Zhu <lipeng.zhu@intel.com>
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.
We discussed earlier that it's better to make functions brand neutral, like using server_
prefix instead of valkey_
, but server_fsync
would be a bit non-unique maybe. I don't mind that we keep valkey
here for now.
Can you update the top comment to list all the names you search-replaced using regex? (If it's not too much...) [Edit] I edited the top comment a little and used it as the commit message when merging.
Thanks!
This patch try to do following things: 1. Rename `redis_*` and `REDIS_*` macros defined in config.h to `valkey_*`, `VALKEY_*` and update associated used files. (`redis_fstat`, `redis_fsync`, `REDIS_THREAD_STACK_SIZE`, etc.) 2. Remove the leading double underscore for guard macro in config.h. --------- Signed-off-by: Lipeng Zhu <lipeng.zhu@intel.com>
This patch try to do following things:
redis_*
andREDIS_*
macros defined in config.h tovalkey_*
,VALKEY_*
and update associated used files. (redis_fstat
,redis_fsync
,REDIS_THREAD_STACK_SIZE
, etc.)