https://bugs.openldap.org/show_bug.cgi?id=8958
--- Comment #38 from Quanah Gibson-Mount quanah@openldap.org --- (In reply to Hallvard Furuseth from comment #36)
On 06.08.2021 13:38, openldap-its@openldap.org wrote:
--- Comment #34 from Howard Chu hyc@openldap.org --- Indexer fix in https://git.openldap.org/openldap/openldap/-/merge_requests/372
I don't think you should change an existing function (pausecheck) to do something completely different. That silently breaks existing code. Renaming it makes sense since the name is confusiong, but use another name for the new functionality. pause_status(), maybe.
Generally I'd expect this type of commentary to be in the MR for historical purposes.