PengZheng commented on PR #657:
URL: https://github.com/apache/celix/pull/657#issuecomment-1734760578

   It's interesting to observe that libxml2 does has its destructor defined:
   
   
https://github.com/GNOME/libxml2/blob/c7ff438b830d8ebeac684f3f8ad3229587c88373/threads.c#L665-L677
   ```C
   #if defined(HAVE_ATTRIBUTE_DESTRUCTOR) && !defined(LIBXML_STATIC) && \
       !defined(_WIN32)
   static void
   ATTRIBUTE_DESTRUCTOR
   xmlDestructor(void) {
       /*
        * Calling custom deallocation functions in a destructor can cause
        * problems, for example with Nokogiri.
        */
       if (xmlFree == free)
           xmlCleanupParser();
   }
   #endif
   ```
   
   Thus there is no need to call `xmlCleanupParser` in our application code, 
unless we are using static libxml2 or custom deallocation function.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@celix.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to