UMutex, static construction & destruction

Description

Figure out a clean pattern for static instances of Mutexes in ICU code.

Followup to and PR #563.
See the discussion there.

Activity

Show:
Andy Heninger
May 10, 2019, 6:03 PM

I am closing this ticket after PR #620, which gets ICU mutex behavior back to where it was with ICU 63, with no static destruction ordering problems and no memory leaks, but with leaking of OS level mutexes.

Follow up work to eliminate the OS mutex leaks will happen under ICU-20588, and will involve yet another refactoring of the ICU mutex code.

 

Jeff Genovy
April 30, 2019, 5:36 PM

Based on the feedback/discussion from Axel on PR 620, we'll need some way to destruct all of the UMutex objects when the ICU library is unloaded – otherwise they will leak the backing OS CriticalSections (which is already causing issues for us).

Andy Heninger
March 25, 2019, 8:33 PM
Fixed
Your pinned fields
Click on the next to a field label to start pinning.

Assignee

Andy Heninger

Reporter

Andy Heninger

Components

Priority

major

Time Needed

Days

Fix versions