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