The problem/use-case that the feature addresses
The method described below: https://github.com/redis/redis/blob/d86cf6610144249f846f3358ea8cc1cf6409b3e8/src/geo.c#L365 initializes a variable named debugmsg with the value 0 https://github.com/redis/redis/blob/d86cf6610144249f846f3358ea8cc1cf6409b3e8/src/geo.c#L368 , and this value does not change, so comparison with the value 0 is redundant. https://github.com/redis/redis/blob/d86cf6610144249f846f3358ea8cc1cf6409b3e8/src/geo.c#L389 The code is unavailable and unused https://github.com/redis/redis/blob/d86cf6610144249f846f3358ea8cc1cf6409b3e8/src/geo.c#L390-L401
Description of the feature
It seems like the value of debugmsg should be changed during code execution, or the check should be excluded
Alternatives you've considered
--
Additional information
debugmsg should be linked to Lua Debugger
Comment From: sundb
@LM4O322 thx, It's just a debug switch (for debugging logs) and won't be dynamically modified. A better approach would be to use a #define, but I prefer to touch it.