WIP:Delete useless checks in confparse.c

config_parse_interval() and config_parse_msec_interval() were checking whether the variable "ok" (a pointer to an int) was null, rather than derefencing it. Both functions are static, and all existing callers pass a valid pointer to those static functions. The callers do check the variables (also confusingly named "ok") whose addresses they pass as the "ok" arguments, so even if the pointer check were corrected to be a dereference, it would be redundant.

Fixes #22103.

added 13 commits

• b1b71ba8...fd437f2a - 11 commits from branch master
• 5494087e - Delete useless checks in confparse.c
• a91f948c - Add tests for #22103

Compare with previous version

merged