https://bugs.openldap.org/show_bug.cgi?id=9726
--- Comment #17 from Howard Chu hyc@openldap.org --- (In reply to Shawn McKinney from comment #16)
(In reply to Howard Chu from comment #15)
syncrepl.c: fprintf(stderr, "merge_state: %s si_syncCookie [%d] %d %s\n", syncrepl.c: fprintf(stderr, "merge_state: %s si_cookieState [%d] %d %s\n",
Instances of fprintf(stderr) in daemon code are anomalies.
and so should converted to use the macro, right? (Admittedly low priority)
When the man page says "debug messages", it's talking about the messages printed to stderr.
It is talking about messages printed to stderr because of the debuglevel setting. it is not talking about the anomalous fprintf(stderr) messages scattered thru daemon code that don't belong there.
So, this means we now have two mechanisms that do the same thing. To prevent (the terminology) confusion, the old mechanism, debuglevel, should be deprecated IMO.
No. The fprintf(stderr) instances that you pointed out (e.g. in syncrepl.c) are behind testing #ifdef's that aren't built by default. For the purposes of this discussion, they don't exist.
There are not two mechanisms that do the same thing.