From 41a02591b9d199a82bb24c23f3828102375137de Mon Sep 17 00:00:00 2001 From: Robin Haberkorn Date: Mon, 3 Jul 2023 04:27:49 +0300 Subject: introduced TECO_DEBUG_CLEANUP to mark destructors that should only be used for debug builds * There is cleanup that is not strictly necessary, because it only frees memory which is freed on program termination anyway. * However, it helps to explicitly free everything for debugging memory leaks via Valgrind. * The new macro reduces the number of #ifdef statements. * On NDEBUG, the code of these functions will still be eliminated. * If functions are referenced only from the destructor, there will be no unused function warnings, even in NDEBUG. --- TODO | 4 ---- 1 file changed, 4 deletions(-) (limited to 'TODO') diff --git a/TODO b/TODO index e0d22e8..ee0cd91 100644 --- a/TODO +++ b/TODO @@ -473,10 +473,6 @@ Optimizations: * Add a configure-switch for LTO (--enable-lto). * undo__teco_interface_ssm() could always include the check for teco_current_doc_must_undo(). - * Define TECO_DESTRUCTOR depending on NDEBUG instead - of ifdefing all __attribute__((destructors)). - This also avoids warning for symbols referenced only in - the constructors. Documentation: * Code docs (Doxygen). It's slowly getting better... -- cgit v1.2.3