mirrors
/
redis
mirror of https://mirror.osredm.com/root/redis.git
2
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Apply suggestions from code review 

* apply comment suggestions

Co-authored-by: Oran Agra <oran@redislabs.com>
This commit is contained in:
kei-nan 2025-02-05 10:01:24 +02:00 committed by YaacovHazan
parent f164012c19
commit 83c0348553
1 changed files with 2 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/module.c
View File

@ -13286,11 +13286,8 @@ int RM_RegisterNumericConfig(RedisModuleCtx *ctx, const char *name, long long de
/* Applies all default configurations for the parameters the module registered.
* Only call this function if the module would like to make changes to the
* configuration values before the actual values are applied by RedisModule_LoadConfigs.
* Otherwise continue calling RedisModule_LoadConfigs, it should already set the default values if needed.
* This sets an ordering for the possible source of a configuration value:
* 1. config values - e.g from the config file or CONFIG during loadex
* 2. module values - e.g from the module arguments
* 3. default values - if no other value was set
* Otherwise it's sufficient to call RedisModule_LoadConfigs, it should already set the default values if needed.
* This makes it possible to distinguish between default values and user provided values and apply other changes between setting the defaults and the user values.```
* This will return REDISMODULE_ERR if it is called:
* 1. outside RedisModule_OnLoad
* 2. more than once