ICU4C Locale (locid.h) needs std::move operators

Description

The `Locale` class in ICU4C is rather nontrivial in its copy assignment and copy constructor (several mallocs and frees). It seems like a very good candidate for upgrading to C++11 with std::move operators (move constructor and move assignment).

Instances of this class are copied around rather freely within the number formatting code, leading to a lot of unnecessary work.

Activity

Show:
TracBot
July 1, 2018, 12:05 AM
Trac Comment 1 by —2018-03-21T18:24:47.019Z

See the proposed patch in and my comment 22 there.

Fixed

Assignee

Fredrik Roubert

Reporter

Shane Carr

Components

Labels

None

Reviewer

None

Priority

medium

Time Needed

None

Fix versions

Configure