Uploaded image for project: 'ICU'
  1. ICU
  2. ICU-13645

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

    Details

    • Type: New Feature/API (Proposal Required)
    • Status: Done (View workflow)
    • Priority: medium
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 63.1
    • Component/s: locale_id
    • Labels:
      None
    • tracCc:
      markus,roubert,shane
    • tracOwner:
      roubert
    • tracProject:
      ICU4C
    • tracReporter:
      shane
    • tracStatus:
      accepted

      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.

        Attachments

          Issue links

          relates to

          New Feature/API (Proposal Required) - A new API or other feature. The assignee will send a proposal to icu-design@, which must then be approved by ICU-TC. ICU-8324 C++11 move semantics for classes

          • medium -
          • Accepted

            Activity
            Cancel

              People

              • Assignee:
                roubert Fredrik Roubert
                Reporter:
                shane Shane Carr
              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  tracCreated: