[STDCXX-792] __rw_locale::_C_manage() uses mutex based __rw_atomic_preincrement() even if platform supports the native atomic operations

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 4.1.2, 4.1.3, 4.1.4, 4.2.0
• Fix Version/s: 4.2.1
• Component/s: 22. Localization

Environment

All

• Patch Info: Patch Available
• Severity: Inefficiency
• Assignee: Farid Zaripov
• Reporter: Farid Zaripov
• Created: 24/Mar/08 16:24
• Updated: 17/Apr/08 10:05
• Resolved: 25/Mar/08 14:31

Description

In __rw_locale::_C_manage() (src/locale_body.cpp, line 808), the static variable ginit declared as volatile, but __rw_atomic_preincrement() functions, which are implemented using atomic operations, accepting the non-volatile reference in parameter list. As a result the template version __rw_atomic_preincrement<volatile long>() (which is implemented using mutex object) is used.

Issue Links

• relates to: STDCXX-794 The non-template __rw_atomic_xxx() functions should accept the volatile reference